You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by pm...@apache.org on 2018/04/21 20:42:10 UTC

svn commit: r1829747 - /jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPHC4Impl.java

Author: pmouawad
Date: Sat Apr 21 20:42:10 2018
New Revision: 1829747

URL: http://svn.apache.org/viewvc?rev=1829747&view=rev
Log:
Fix regression on Redirect Handling

Modified:
    jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPHC4Impl.java

Modified: jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPHC4Impl.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPHC4Impl.java?rev=1829747&r1=1829746&r2=1829747&view=diff
==============================================================================
--- jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPHC4Impl.java (original)
+++ jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPHC4Impl.java Sat Apr 21 20:42:10 2018
@@ -1051,9 +1051,6 @@ public class HTTPHC4Impl extends HTTPHCA
                 }
                 builder.setDefaultCredentialsProvider(credsProvider);
             }
-            if(getAutoRedirects()) {
-                builder.disableRedirectHandling();
-            }
             builder.disableContentCompression().addInterceptorLast(RESPONSE_CONTENT_ENCODING);
             if(BASIC_AUTH_PREEMPTIVE) {
                 builder.addInterceptorFirst(PREEMPTIVE_AUTH_INTERCEPTOR);
@@ -1152,6 +1149,7 @@ public class HTTPHC4Impl extends HTTPHCA
             rCB.setConnectTimeout(cto);
         }
     
+        rCB.setRedirectsEnabled(getAutoRedirects());
         rCB.setMaxRedirects(HTTPSamplerBase.MAX_REDIRECTS);
         rCB.setRedirectsEnabled(getAutoRedirects());
         httpRequest.setConfig(rCB.build());