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 12b0cfe
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 12b0cfe

Please sign in to comment.