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

Add groupsClaimMapping to the OIDC connector #1776

Closed
wants to merge 1 commit into from

Conversation

Lemmons
Copy link
Contributor

@Lemmons Lemmons commented Aug 7, 2020

The groupsClaimMapping setting allows one to specify which claim to pull group information from the OIDC provider. Previously it assumed group information was always in the "groups" claim, but that isn't the case for many OIDC providers (such as AWS Cognito using the "cognito:groups" claim instead)

The groupsClaimMapping setting allows one to specify which claim to pull
group information from the OIDC provider.  Previously it assumed group
information was always in the "groups" claim, but that isn't the case
for many OIDC providers (such as AWS Cognito using the "cognito:groups"
claim instead)

Signed-off-by: Scott Lemmon <[email protected]>
@Lemmons Lemmons force-pushed the oidc-group-claims-mapping branch from 9c64a9a to bdd56f8 Compare August 7, 2020 22:14
@Lemmons
Copy link
Contributor Author

Lemmons commented Aug 7, 2020

This is related to #1065

@Lemmons
Copy link
Contributor Author

Lemmons commented Aug 7, 2020

Looks like #1631 implemented this same logic back in Jan, but is still waiting for a review.

@sagikazarmark
Copy link
Member

I'm fine with resubmitting it.

Please see #1777 before reviewing and merging this.

xtremerui pushed a commit to concourse/dex that referenced this pull request Aug 11, 2020
add tests for groups key mapping

Signed-off-by: Rui Yang <[email protected]>
@sagikazarmark
Copy link
Member

Fixed in #1634

elffjs pushed a commit to DIMO-Network/dex that referenced this pull request Jun 27, 2022
add tests for groups key mapping

Signed-off-by: Rui Yang <[email protected]>
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