Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Support Organization Name #884

Merged
merged 1 commit into from
Jul 14, 2023
Merged

Support Organization Name #884

merged 1 commit into from
Jul 14, 2023

Conversation

frederikprijck
Copy link
Member

@frederikprijck frederikprijck commented Jul 14, 2023

Changes

Adds support for the organization name by extending the ID Token validation as follows:

  • If organization is set and has the org_ prefix, we expect an org_id claim
  • If organization is set and does not have the org_ prefix, we expect an org_name claim

Testing

Please describe how this can be tested by reviewers. Be specific about anything not tested and reasons why. If this library has unit and/or integration testing, tests should be added for new functionality and existing tests should complete without errors.

  • This change adds unit test coverage
  • This change adds integration test coverage

Checklist

@frederikprijck frederikprijck requested a review from a team as a code owner July 14, 2023 11:48
@frederikprijck frederikprijck merged commit 1935ff8 into master Jul 14, 2023
@frederikprijck frederikprijck deleted the feat/sdk-4392 branch July 14, 2023 13:48
@frederikprijck frederikprijck mentioned this pull request Jul 18, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants