diff options
author | Mårten Nordheim <marten.nordheim@qt.io> | 2020-05-05 15:41:35 +0200 |
---|---|---|
committer | Mårten Nordheim <marten.nordheim@qt.io> | 2020-05-19 09:18:40 +0000 |
commit | aed7a65b8b0870d8a019f7917888587de2a66303 (patch) | |
tree | 5574e061a1ac1f4068fe35c6d066113fb57bb9e1 /src/network/kernel | |
parent | 19b6ae6637bde73ec508b6d1d29f3233e2c4e2d4 (diff) |
QAuthenticator: Reprioritize authentication methods
The addition of Negotiate to QAuthenticator caused a soft sort of
regression in environments where there is key distribution center
set up. This was due to how QAuthenticator works in that it will
simply prefer the latest entry in the enum, which now was Negotiate,
which sadly wasn't really a valid option in that situation. And it is
not smart enough to fall back and try another method.
To work around this issue we re-order the enum to prioritize
authentication methods to restore previous behavior.
Note that Negotiate is still preferred over Basic and None because
they're not likely to appear together.
Pick-to: 5.15
Task-number: QTBUG-83905
Change-Id: Ic528318b6b711aa04d42a86c684452bb6ebde112
Reviewed-by: Timur Pocheptsov <timur.pocheptsov@qt.io>
Diffstat (limited to 'src/network/kernel')
-rw-r--r-- | src/network/kernel/qauthenticator_p.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/network/kernel/qauthenticator_p.h b/src/network/kernel/qauthenticator_p.h index e201d22650..99dfdbda2c 100644 --- a/src/network/kernel/qauthenticator_p.h +++ b/src/network/kernel/qauthenticator_p.h @@ -71,7 +71,7 @@ class QGssApiHandles; class Q_AUTOTEST_EXPORT QAuthenticatorPrivate { public: - enum Method { None, Basic, Ntlm, DigestMd5, Negotiate }; + enum Method { None, Basic, Negotiate, Ntlm, DigestMd5, }; QAuthenticatorPrivate(); ~QAuthenticatorPrivate(); |