Skip to content
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

Question on refresh token scopes #1117

Open
Misosooup opened this issue May 28, 2020 · 1 comment
Open

Question on refresh token scopes #1117

Misosooup opened this issue May 28, 2020 · 1 comment
Milestone

Comments

@Misosooup
Copy link

At the moment, refresh token scopes are returned base on the access token scopes set.

When a user request for a new access token via the refresh_token grant type and he/she requested lesser scope than what the original access token has. Should the refresh token have the original scopes or should the refresh token has the new scopes requested.

  1. If the refresh token has the new scopes requested, does this mean that eventually, they will run out of scopes if they keep requesting lesser scopes?
  2. Should the refresh token keep the original scopes? This would mean that an access token returned would have different scopes as to what is stored in the refresh token, and the next request to get a new access token may result in more scopes than the current access token.

Can someone please enlighten me on this issue?

I have read the RFC docs and there is a point that states
If a new refresh token is issued, the refresh token scope MUST be identical to that of the refresh token included by the client in the request.

@Sephster
Copy link
Member

Sephster commented Aug 3, 2020

It looks like we should be issuing the refresh token with the same scopes as the original, regardless of what scopes were requested.

I think this should probably be changes to better match the spec. Thanks for flagging.

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

No branches or pull requests

2 participants