Skip to content

Extend BearerOnlyAuthenticator with option to fetch roles from userinfo endpoint#10230

Merged
nielsm5 merged 5 commits intomasterfrom
issue/10224-userinfo-endpoint-in-bearer-only-authenticator
Jan 8, 2026
Merged

Extend BearerOnlyAuthenticator with option to fetch roles from userinfo endpoint#10230
nielsm5 merged 5 commits intomasterfrom
issue/10224-userinfo-endpoint-in-bearer-only-authenticator

Conversation

@evandongen
Copy link
Contributor

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)

@evandongen evandongen linked an issue Jan 7, 2026 that may be closed by this pull request
Copy link
Contributor Author

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

@evandongen evandongen requested review from Matthbo and nielsm5 January 7, 2026 13:37
Copy link
Member

@nielsm5 nielsm5 left a comment

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).

@Matthbo
Copy link
Member

Matthbo commented Jan 7, 2026

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
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 8, 2026

@nielsm5 nielsm5 enabled auto-merge (squash) January 8, 2026 13:24
@nielsm5 nielsm5 merged commit 509e1e5 into master Jan 8, 2026
22 of 23 checks passed
@nielsm5 nielsm5 deleted the issue/10224-userinfo-endpoint-in-bearer-only-authenticator branch January 8, 2026 14:12
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.

OIDC from separate frontend application support

3 participants