Extend BearerOnlyAuthenticator with option to fetch roles from userinfo endpoint by evandongen · Pull Request #10230 · frankframework/frankframework
Changes
Pull Request Checklist
Title
- Title expresses the business value (who benefits + what outcome)
Issues
- Relevant issues linked
Backports
- Backport PRs created (if needed) and linked
Documentation
- FF! Doc updated (user-facing behavior/config)
- FF! Manual updated (if applicable)
- Javadoc updated/generated (developer-facing APIs)
Tests
- Unit tests added/updated
- E2E/Integration tests added/updated (if applicable)
Breaking changes
- Breaking change recorded in markdown file
- Migration notes included (if needed)
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Het langste hiermee bezig geweest om te bepalen welke settings ik nou in de json miste tov van mijn keycloak instance :-X
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it looks ok, although I don't fully understand the test setup (yet).
Maybe I can build this PR as a docker image so I can deploy the project with these changes and see if that works
…ly-authenticator # Conflicts: # test/src/test/java/org/frankframework/runner/KeycloakBearerOnlyAuthenticatorIntegrationTest.java
nielsm5
deleted the
issue/10224-userinfo-endpoint-in-bearer-only-authenticator
branch
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