-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
UI: JWT Auth Browser Popup warning. Resolves: #10753 #17009
UI: JWT Auth Browser Popup warning. Resolves: #10753 #17009
Conversation
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.
Thanks for tackling this! Just a couple of issues with the changelog entry but the message update looks good!
hey @chelshaw do you know if this will make it out with the next 1.12 release - I believe it's not made it out with the 1.12.0 GA? |
@aphorise - It looks like this got lost in the shuffle, sorry about that! The CircleCI tests aren't running which is prohibiting this PR from merging. I'm wondering if it could be due to following the fork instead of the upstream repo CircleCi troubleshooting @hsimon-hashicorp - any thoughts? |
Closing in favour of #18787 |
Pull request was closed
Clarify warning and message as per community suggestion (#10753) to be more explicit about blocked pop-ups.