-
Notifications
You must be signed in to change notification settings - Fork 4.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PKI role create: fix issue with setting default key_bits on init and when key_type changes #17613
Conversation
@@ -10,10 +10,10 @@ | |||
/> | |||
<Toolbar> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These changes fix something that seemed to have broken after the ember upgrade.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes I had to refactor the ToolbarLink
since passing positional params into LinkTo
is no longer supported
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! I think it was a good move to bring this into a component. Thanks
…when key_type changes (hashicorp#17613) * working * fix issue with signature bits * fix ember upgrade change * clean up * fix signature bits to number * default value in model * fix language
There was an issue where the
keyBit
value was not being set on init nor was it being set whenkeyType
changed, even though the dropdown options were changing.I pulled all the conditional options work out of the model, and moved it all into a component. This should help clarify what's going on, and move semi-complication logic work out of the model. Also, it will help organize testing.
keyparams.mov
In the API you can see the default key_bits set for each key_type.
And the default key_type is
rsa
. You can test this in the current PKI. Even though we don't send this default value in the old PKI, the read view shows rsa and 2048.