Skip to content

Fix usage of OktaJwtAuthenticationConverter to use constructor with props#752

Merged
arvindkrishnakumar-okta merged 1 commit intookta:fix_oktaJwtAuthenticationConverter_usagefrom
mpanik:fix-usage-of-jwt-auth-converter
Feb 3, 2025
Merged

Fix usage of OktaJwtAuthenticationConverter to use constructor with props#752
arvindkrishnakumar-okta merged 1 commit intookta:fix_oktaJwtAuthenticationConverter_usagefrom
mpanik:fix-usage-of-jwt-auth-converter

Conversation

@mpanik
Copy link
Copy Markdown

@mpanik mpanik commented Dec 22, 2024

Updated the autoconfiguration to use the OktaJwtAuthenticationConverter constructor that accepts OktaOAuth2Properties instead of the one that accepts a group name. This change allows the authoritiesClaimName property to be used for mapping authorities from the "permissions" claim, ensuring proper configuration and functionality.

…rops

Updated the autoconfiguration to use the OktaJwtAuthenticationConverter constructor that accepts OktaOAuth2Properties instead of the one that accepts a group name. This change allows the authoritiesClaimName property to be used for mapping authorities from the "permissions" claim, ensuring proper configuration and functionality.
@arvindkrishnakumar-okta
Copy link
Copy Markdown
Contributor

@mpanik Thanks for the PR!

If not already, can you submit a CLA?

@mpanik
Copy link
Copy Markdown
Author

mpanik commented Jan 9, 2025

@arvindkrishnakumar-okta signed and sent.

@arvindkrishnakumar-okta arvindkrishnakumar-okta changed the base branch from master to fix_oktaJwtAuthenticationConverter_usage February 3, 2025 16:04
@arvindkrishnakumar-okta arvindkrishnakumar-okta merged commit a72c81b into okta:fix_oktaJwtAuthenticationConverter_usage Feb 3, 2025
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