Bug 1410257 - make non-connection-based connections restartable; r=mayhemer draft
authorLiang-Heng Chen <xeonchen@gmail.com>
Thu, 26 Oct 2017 14:23:05 +0800
changeset 686653 9660d322ad5b44bdbdb5da021ee4249e5e9de5d8
parent 686540 64bab5cbb9b63808d04babfbcfba3175fd99f69d
child 737419 b36199276a3ea2060597076fd0498858241decd7
push id86237
push userbmo:xeonchen@mozilla.com
push dateThu, 26 Oct 2017 06:25:00 +0000
reviewersmayhemer
bugs1410257
milestone58.0a1
Bug 1410257 - make non-connection-based connections restartable; r=mayhemer MozReview-Commit-ID: 1hEk7mT6mzZ
netwerk/protocol/http/nsHttpChannelAuthProvider.cpp
--- a/netwerk/protocol/http/nsHttpChannelAuthProvider.cpp
+++ b/netwerk/protocol/http/nsHttpChannelAuthProvider.cpp
@@ -830,17 +830,17 @@ nsHttpChannelAuthProvider::GetCredential
     }
 
     mConnectionBased = !!(authFlags & nsIHttpAuthenticator::CONNECTION_BASED);
 
     // It's legal if the peer closes the connection after the first 401/7.
     // Making the connection sticky will prevent its restart giving the user
     // a 'network reset' error every time.  Hence, we mark the connection
     // as restartable.
-    mAuthChannel->ConnectionRestartable(mConnectionBased && !authAtProgress);
+    mAuthChannel->ConnectionRestartable(!authAtProgress);
 
     if (identityInvalid) {
         if (entry) {
             if (ident->Equals(entry->Identity())) {
                 if (!identFromURI) {
                     LOG(("  clearing bad auth cache entry\n"));
                     // ok, we've already tried this user identity, so clear the
                     // corresponding entry from the auth cache.