Skip to content

Commit

Permalink
Remove scope from the token refresh request as it is redundant
Browse files Browse the repository at this point in the history
Scope is a valid parameter for the Refresh Token request (Sectiom 6 of
RFC 6749), however it's optional and when ommitted is treated as equal
to the scope originally granted by the resource owner. Since the
indented behavior of this convenience method is to create a token
refresh with the full scope, it's redundant to include.

Related to b5870c0 but slightly
different reason.
  • Loading branch information
WilliamDenniss committed Sep 21, 2018
1 parent fadb76d commit e7077f1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Source/OIDAuthState.m
Original file line number Diff line number Diff line change
Expand Up @@ -414,7 +414,7 @@ - (OIDTokenRequest *)tokenRefreshRequestWithAdditionalParameters:
redirectURL:nil
clientID:_lastAuthorizationResponse.request.clientID
clientSecret:_lastAuthorizationResponse.request.clientSecret
scope:_lastAuthorizationResponse.request.scope
scope:nil
refreshToken:_refreshToken
codeVerifier:nil
additionalParameters:additionalParameters];
Expand Down

0 comments on commit e7077f1

Please sign in to comment.