diff options
| author | Juha Vuolle <juha.vuolle@qt.io> | 2024-11-15 09:38:54 +0200 |
|---|---|---|
| committer | Juha Vuolle <juha.vuolle@qt.io> | 2024-11-21 12:17:28 +0200 |
| commit | 97b0865a0565bd7ff6bfde055510d5e899de6d46 (patch) | |
| tree | 6b57686ffa362ca256b829a177cacc4ceb02829a /src/oauth/qoauth2authorizationcodeflow.cpp | |
| parent | 24c3eff4c96a8a39b760e958513612c4d85ceffe (diff) | |
Remove 'state' parameter from extraTokens
State parameter is consumed by the flow class and should not be in the
extraTokens property. The documentation of the property is not clear
on this, but this would be in line how access token response populates
the extraTokens: only leave tokens that are considered extra.
[ChangeLog][OAuth2] received state parameter is no longer provided in
extraTokens property.
Change-Id: I157bc33f4caf3c5d8bde538d33f257b20d2593eb
Reviewed-by: Ivan Solovev <ivan.solovev@qt.io>
Reviewed-by: Jesus Fernandez <jsfdez@gmail.com>
Diffstat (limited to 'src/oauth/qoauth2authorizationcodeflow.cpp')
| -rw-r--r-- | src/oauth/qoauth2authorizationcodeflow.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/oauth/qoauth2authorizationcodeflow.cpp b/src/oauth/qoauth2authorizationcodeflow.cpp index a5e3380..fdb476a 100644 --- a/src/oauth/qoauth2authorizationcodeflow.cpp +++ b/src/oauth/qoauth2authorizationcodeflow.cpp @@ -134,6 +134,7 @@ void QOAuth2AuthorizationCodeFlowPrivate::_q_handleCallback(const QVariantMap &d QVariantMap copy(data); copy.remove(Key::code); + copy.remove(Key::state); setExtraTokens(copy); q->requestAccessToken(code); } |