Skip to content

Record access token scope in the grant response #48153

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sberyozkin
Copy link
Member

@sberyozkin sberyozkin commented May 30, 2025

Fixes #48152

The main update is here - if it is authorization code flow token, check the scope from the authorization code flow response as well. This update is really relevant with social providers like GitHub when the token is binary, if it is JWT then JSON with roles/permissions is checked a few lines earlier - which is where I had to fix NPE - until now, we have only enforced @Authenticated at the point of login with social providers.

Updated tests, confirmed I can use for example, @PermissionsAllowed("user:email") with the GitHub login.

Copy link

quarkus-bot bot commented May 30, 2025

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit dbcbe87.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Retain access token scope from the authorization code flow response
1 participant