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

OIDC bug fix role_type is not allowed #15666

Merged
merged 7 commits into from
May 31, 2022
Merged

Conversation

Monkeychip
Copy link
Contributor

This essentially backports a fix that was originally backported only to 1.10 (see here) with the thinking the bug did not exist before that binary. Turns out it did so we are backporting only the specific lines that fix this bug. Given this release branch is not much different than 1.11, we wanted to keep the scope to the fix and not the other refactoring that was done in the original PR.

@Monkeychip Monkeychip added ui bug Used to indicate a potential bug pr/no-changelog labels May 27, 2022
@Monkeychip Monkeychip added this to the 1.9.7 milestone May 27, 2022
Monkeychip and others added 5 commits May 27, 2022 12:21
Fix automatic linting done by the editor before you're allowed to push.
Fix linting
linting cont.
Copy link
Contributor

@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.

Looks good! Let's go ahead and add a changelog for the fix though so that folks who run into the issue on other 1.9 versions can see which version we have a fix in

@Monkeychip Monkeychip merged commit ad0b820 into release/1.9.x May 31, 2022
@Monkeychip Monkeychip deleted the ui/oidc-bug-fix branch May 31, 2022 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Used to indicate a potential bug ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants