diff options
| author | Ivan Solovev <ivan.solovev@qt.io> | 2024-12-16 16:32:24 +0100 |
|---|---|---|
| committer | Ivan Solovev <ivan.solovev@qt.io> | 2024-12-19 09:31:04 +0100 |
| commit | b05d99c937f7ea9226314eaaae628cdfdda36bbe (patch) | |
| tree | 93cde0de798c07f0bd1955c575e2a0bbcf3577b4 /src/oauth/qoauth2authorizationcodeflow_p.h | |
| parent | 82e516de8196b04f3240fa910b5997a076ce8e1e (diff) | |
Move tokenUrl property to QAbstractOAuth2
The property is common for both authorization code and device flows, so
it makes sense to move it to the base class. And more broadly speaking,
most OAuth2 flows use a token endpoint.
One problem is that QOAuth2AuthorizationCodeFlow already has an
accessTokenUrl property which is used for the same purpose. This patch
implements this property in termes of the base tokenUrl property. The
follow-up commit will deprecate the QOAuth2AuthorizationCodeFlow
property and mark it for removal in Qt 7.
There were no tests for the accessTokenUrl property and the related
signal, so create a new test that checks both old and new properties.
[ChangeLog][QAbstractOAuth2] Added tokenUrl property that holds the
token endpoint URL.
Task-number: QTBUG-132106
Pick-to: 6.9
Change-Id: I4e680e1013093041dcd3fa4f06e24b83cec83fc3
Reviewed-by: Juha Vuolle <juha.vuolle@qt.io>
Diffstat (limited to 'src/oauth/qoauth2authorizationcodeflow_p.h')
| -rw-r--r-- | src/oauth/qoauth2authorizationcodeflow_p.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/oauth/qoauth2authorizationcodeflow_p.h b/src/oauth/qoauth2authorizationcodeflow_p.h index f5d143c..e80cc82 100644 --- a/src/oauth/qoauth2authorizationcodeflow_p.h +++ b/src/oauth/qoauth2authorizationcodeflow_p.h @@ -50,7 +50,6 @@ public: quint8 pkceVerifierLength = 43; // RFC 7636 Section 4.1 QByteArray pkceCodeVerifier; - QUrl accessTokenUrl; QPointer<QNetworkReply> currentReply; }; |