Fix cached scopes when using detailed response mode #824
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes
This PR ensures that the scopes as returned from the token endpoint are recorded in the cache by
handleRedirectCallback
. This fixes an issue where usinggetTokenSilently({ detailedResponse: true })
immediately after a login would yield noscope
property, despite being returned by the endpoint, but are then populated later after a page refresh (thanks tocheckSession
).The PR also addresses a minor issue where
detailedResponse
was being sent as a custom property to the/authorize
and/oauth/token
endpoints.References
Fixes #823
Testing
Checklist