Uh oh!
There was an error while loading. Please reload this page.
Implement Role Mapping on login - #2534
Merged
Merged
Conversation
- maintains a association between a claim and webAPI role. - Can be maintained via CSV load or indiviual REST endpionts - Login flow updated to synchronize mapped roles on login. - Added tests for add/delete/overwrite and validation rules.
chrisknoll
commented
Jul 15, 2026
CollaboratorAuthor
In addition, I'm attempting to fix the authentication issues due to the OIDC re-work from #2533 |
chrisknollforce-pushed
the
cknoll/role-group-mapping
branch
from
July 15, 2026 14:21
aafbaec to
55630c1Compare…y to identify guarded.
chrisknoll
commented
Jul 15, 2026
CollaboratorAuthor
I disabled the endpoint guard tests because the test showed these endpints not guarded: Some of these can be removed, some of these could be marked as unguarded. Need further review. |
Uh oh!
There was an error while loading. Please reload this page.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
New feature: External Role Mapping:
Legacy tables have been moved:
ALTER TABLE ${ohdsiSchema}.sec_role_group RENAME TO sec_group_role_import;
And renamed constraints to match table names.