diff options
author | Shane Kearns <ext-shane.2.kearns@nokia.com> | 2012-03-22 15:10:26 +0000 |
---|---|---|
committer | Qt by Nokia <qt-info@nokia.com> | 2012-03-23 11:45:25 +0100 |
commit | 3c3ea9a869086ebb02a5d0c475754f6a184a50ab (patch) | |
tree | ff9ab96772cf1e1589d3c0cbb671ee15cc06afa7 /src/network | |
parent | b78e355fe5d786710727b89ee5931bea470167d5 (diff) |
Fix performance problem with NTLM auth in http POST
A combination of other fixes resulted in authentication credentials
with a blank username & password being incorrectly added to the
cache only for NTLM.
This caused authentication to be attempted with blank credentials
first (which would fail), before the authenticationRequired
signal was emitted.
It caused a performance issue because of the extra 2 requests to the
server (and possibly delays inserted by the server following a failed
login)
Change-Id: Ic588a20cfe7c24d5e60cd384caff0673a587e484
Reviewed-by: Martin Petersson <Martin.Petersson@nokia.com>
Diffstat (limited to 'src/network')
-rw-r--r-- | src/network/kernel/qauthenticator.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/network/kernel/qauthenticator.cpp b/src/network/kernel/qauthenticator.cpp index eef2a7fa76..43b3618ea2 100644 --- a/src/network/kernel/qauthenticator.cpp +++ b/src/network/kernel/qauthenticator.cpp @@ -380,6 +380,8 @@ void QAuthenticatorPrivate::parseHttpResponse(const QList<QPair<QByteArray, QByt break; case Ntlm: // #### extract from header + if (user.isEmpty() && password.isEmpty()) + phase = Done; break; case DigestMd5: { this->options[QLatin1String("realm")] = realm = QString::fromLatin1(options.value("realm")); |