Don't enforce connection reuse on schemes other than NTLM

Author: Stephane Landelle

providers/netty/src/main/java/org/asynchttpclient/providers/netty/handler/HttpProtocol.java

@@ -207,11 +207,19 @@ private boolean exitAfterHandling401(//
                 if (!wwwAuthHeaders.contains("Kerberos") && (isNTLM(wwwAuthHeaders) || negociate)) {
                     // NTLM
                     newRealm = ntlmChallenge(wwwAuthHeaders.get(0), request, proxyServer, request.getHeaders(), realm, future, false);
+
+                    // don't forget to reuse channel: NTLM authenticates a connection
+                    future.setReuseChannel(true);
+
                 } else if (negociate) {
                     newRealm = kerberosChallenge(channel, wwwAuthHeaders, request, proxyServer, request.getHeaders(), realm, future, false);
                     // SPNEGO KERBEROS
                     if (newRealm == null)
                         return true;
+                    else
+                        // don't forget to reuse channel: KERBEROS authenticates a connection
+                        future.setReuseChannel(true);
+
                 } else {
                     newRealm = new Realm.RealmBuilder()//
                             .clone(realm)//
@@ -230,8 +238,6 @@ private boolean exitAfterHandling401(//
                 Callback callback = new Callback(future) {
                     public void call() throws IOException {
                         channelManager.drainChannel(channel, future);
-                        // don't forget to reuse channel: NTLM authenticates a connection
-                        future.setReuseChannel(true);
                         requestSender.sendNextRequest(nextRequest, future);
                     }
                 };