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

Fix privileged business user roles issue #5524

Merged
merged 2 commits into from
Mar 5, 2024

Conversation

Yoshani
Copy link
Contributor

@Yoshani Yoshani commented Feb 19, 2024

Proposed changes in this pull request

This PR introduces a config for fixing an issue which is observed in products using old runtime where a privileged business user cannot properly access the console due to not receiving the admin scopes.
This is a temporary fix for the related issue until the new runtime is introduced to all products. The config is not related to product IS, thus it is disabled by default.
The config is as below.

[legacy_claims]
allow_legacy_role_claim_behaviour=false

When should this PR be merged

Once the IS 7.0 release is done.

@Yoshani Yoshani force-pushed the fix-busi-admin-console-login branch 2 times, most recently from 107e95a to 78bdaa2 Compare February 19, 2024 08:58
@Yoshani Yoshani force-pushed the fix-busi-admin-console-login branch 2 times, most recently from 9a08524 to d2f746f Compare February 26, 2024 12:38
@Yoshani Yoshani force-pushed the fix-busi-admin-console-login branch from d2f746f to fe2e0eb Compare March 5, 2024 04:13
@jenkins-is-staging
Copy link

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/8151649541

@jenkins-is-staging
Copy link

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/8151649541
Status: success

Copy link

@jenkins-is-staging jenkins-is-staging left a comment

Choose a reason for hiding this comment

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

Approving the pull request based on the successful pr build https://github.com/wso2/product-is/actions/runs/8151649541

@Yoshani Yoshani merged commit 8dcd10d into wso2:master Mar 5, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants