Skip to content

Pesomka/extra scopes#266

Merged
SomkaPe merged 4 commits intodevfrom
pesomka/extra_scopes
Aug 17, 2020
Merged

Pesomka/extra scopes#266
SomkaPe merged 4 commits intodevfrom
pesomka/extra_scopes

Conversation

@SomkaPe
Copy link
Copy Markdown
Contributor

@SomkaPe SomkaPe commented Aug 4, 2020

Extra scope to consent during authorization request

Copy link
Copy Markdown
Contributor

@sangonzal sangonzal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems that extraScopesToConsent just provides another interface to add scopes, but ultimately they just get appended to the scopes that were passed in as part of the request. It seems like the key distinction is that they should go after the scopes passed in the request. In this case, the customer receives an access token for scopes from request, and a refresh token that works for both scopes and extraScopesToConsent, where they have to acquireTokeSilent to get an access token for extraScopesToConsent.

Java Set does not guarantee order. If server is relying on the order of the scopes to make this distinction (I'm not sure that's the case, just assuming that from this implementation), we should probably use a data structure that guarantees order (such as TreeSet).

@SomkaPe
Copy link
Copy Markdown
Contributor Author

SomkaPe commented Aug 11, 2020

good point about order of scopes, but it should not matter because it is used just for authorization request, during token request (with auth code) scope for access token is specified

@SomkaPe SomkaPe merged commit 7213c0c into dev Aug 17, 2020
@siddhijain siddhijain deleted the pesomka/extra_scopes branch January 19, 2022 18:27
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.

3 participants