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/OIDC auth bug for hcp namespace flag into release/1.9.x #16909

Conversation

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

Backport

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

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


Fixes the afterModel() hook so that it can handle both when a namespace comes in as a param from the cluster: namespaceQueryParam = 'admin' and when it's attached to the state from the route params: state="st_z6y8L12PndJ33FmW1Ls2,ns=admin"

The namespace passed via state will take precedence


Overview of commits

@hc-github-team-secure-vault-core hc-github-team-secure-vault-core force-pushed the backport/ui/VAULT-7938/oidc-auth-bug-for-hcp-namespace-flag/namely-loving-troll branch from 0e9053a to e8efbb9 Compare August 26, 2022 17:09
@hashishaw hashishaw added this to the 1.9.9 milestone Aug 26, 2022
@hashishaw hashishaw added ui backport bug Used to indicate a potential bug labels Aug 26, 2022
Copy link
Collaborator

@hashishaw hashishaw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@hashishaw hashishaw enabled auto-merge (squash) August 26, 2022 17:26
@hashishaw hashishaw merged commit 9c11f0a into release/1.9.x Aug 26, 2022
@hellobontempo hellobontempo deleted the backport/ui/VAULT-7938/oidc-auth-bug-for-hcp-namespace-flag/namely-loving-troll branch August 26, 2022 18:00
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.

None yet

3 participants