Skip to content
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

Backport of UI cluster unseal bug into release/1.13.x #20909

Conversation

hc-github-team-secure-vault-core
Copy link
Collaborator

Backport

This PR is auto-generated from #20897 to be assessed for backporting due to the inclusion of the label backport/1.13.x.

The below text is copied from the body of the original PR.


It was discovered that the secret_shares and secret_threshold properties are no longer allowed to be passed when unsealing a cloud seal type cluster. These properties were being passed in the UI causing the unseal to fail. Removing these properties from the payload for seal types other than shamir allows the cluster to be unsealed as expected.

Error before fix:
image

After fix:

Screen.Recording.2023-05-31.at.2.16.46.PM.mov

Resolves #20644


Overview of commits

@hc-github-team-secure-vault-core hc-github-team-secure-vault-core force-pushed the backport/ui/VAULT-16511/auto-unseal-bug/positively-assuring-adder branch from 6daa1bd to 5d8d283 Compare May 31, 2023 20:21
@hc-github-team-secure-vault-core hc-github-team-secure-vault-core force-pushed the backport/ui/VAULT-16511/auto-unseal-bug/positively-assuring-adder branch from 34fe193 to e0afedb Compare May 31, 2023 20:22
@zofskeez zofskeez added this to the 1.13.3 milestone May 31, 2023
@zofskeez zofskeez added the ui label May 31, 2023
@hellobontempo hellobontempo added backport bug Used to indicate a potential bug labels May 31, 2023
@zofskeez zofskeez merged commit 780f4f4 into release/1.13.x May 31, 2023
@zofskeez zofskeez deleted the backport/ui/VAULT-16511/auto-unseal-bug/positively-assuring-adder branch May 31, 2023 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport bug Used to indicate a potential bug ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants