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

CAMS-446 - Okta ad group bug #930

Merged
merged 4 commits into from
Sep 23, 2024
Merged

CAMS-446 - Okta ad group bug #930

merged 4 commits into from
Sep 23, 2024

Conversation

btposey
Copy link
Contributor

@btposey btposey commented Sep 23, 2024

Purpose

Use both ad_groups and groups claims for groups from Okta users

Major Changes

  • Refactored UserSessionGateway into UserSessionUseCase
  • Merged the ad_groups and groups claims into a single group set.

Testing/Validation

Added tests for unifying the ad_groups and groups claims

btposey and others added 3 commits September 23, 2024 16:14
* Moved UserSessionGateway to use case directory
* Added ad_groups to Okta JWT claims type
* Merged ad_groups and groups claims into a single groups claim

Jira ticket: CAMS-446

Co-authored-by: James Brooks <[email protected]>
Co-authored-by: Brian Posey <[email protected]>,
Jira ticket: CAMS-446

Co-authored-by: Brian Posey <[email protected]>,
Jira ticket: CAMS-446

Co-authored-by: James Brooks <[email protected]>
Co-authored-by: Brian Posey <[email protected]>,
@btposey btposey requested review from a team as code owners September 23, 2024 21:10
Jira ticket: CAMS-446

Co-authored-by: James Brooks <[email protected]>
Co-authored-by: Brian Posey <[email protected]>,
@btposey btposey merged commit 1dad6f1 into main Sep 23, 2024
12 checks passed
@btposey btposey deleted the CAMS-446-okta-ad-group-bug branch September 23, 2024 21:17
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.

2 participants