diff options
| author | Juha Vuolle <juha.vuolle@qt.io> | 2025-01-20 10:22:34 +0200 |
|---|---|---|
| committer | Juha Vuolle <juha.vuolle@qt.io> | 2025-01-24 07:36:27 +0200 |
| commit | 3a0b25172534d178507e80c88d66ad5b6271eb2f (patch) | |
| tree | ccf514f74d80438f21ea9389065dda9c1b7158d3 /src/oauth/qoauth2authorizationcodeflow.cpp | |
| parent | b09f606030465a3ae6d73ec179579209fc564771 (diff) | |
Change '*scopeTokens' type from QStringList to QSet<QByteArray>
This better reflects the scope (token) semantics:
- Token order doesn't matter
- Duplicate tokens don't make sense
- Character encoding is limited to ASCII (albeit there might be
proprietary authorization servers not adhering to this)
Resulted from API-review.
Amends: 571b71763c0485778623139359acc88985099efb
Task-number: QTBUG-132710
Pick-to: 6.9
Change-Id: I781a707fcb3e2f06309e7ea57c5b05203afce03a
Reviewed-by: Marc Mutz <marc.mutz@qt.io>
Diffstat (limited to 'src/oauth/qoauth2authorizationcodeflow.cpp')
| -rw-r--r-- | src/oauth/qoauth2authorizationcodeflow.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/oauth/qoauth2authorizationcodeflow.cpp b/src/oauth/qoauth2authorizationcodeflow.cpp index 3cf61ed..33c47fa 100644 --- a/src/oauth/qoauth2authorizationcodeflow.cpp +++ b/src/oauth/qoauth2authorizationcodeflow.cpp @@ -457,7 +457,7 @@ QUrl QOAuth2AuthorizationCodeFlow::buildAuthenticateUrl(const QMultiMap<QString, p.insert(QtOAuth2RfcKeywords::clientIdentifier, d->clientIdentifier); p.insert(QtOAuth2RfcKeywords::redirectUri, callback()); if (!d->requestedScopeTokens.isEmpty()) - p.insert(QtOAuth2RfcKeywords::scope, d->requestedScopeTokens.join(" "_L1)); + p.insert(QtOAuth2RfcKeywords::scope, d->joinedScope(d->requestedScopeTokens)); p.insert(QtOAuth2RfcKeywords::state, toUrlFormEncoding(state)); if (d->pkceMethod != PkceMethod::None) { p.insert(QtOAuth2RfcKeywords::codeChallenge, d->createPKCEChallenge()); |