You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by se...@apache.org on 2012/11/10 16:30:58 UTC

svn commit: r1407837 - /jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSampleResult.java

Author: sebb
Date: Sat Nov 10 15:30:57 2012
New Revision: 1407837

URL: http://svn.apache.org/viewvc?rev=1407837&view=rev
Log:
Document tweak

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

Modified: jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSampleResult.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSampleResult.java?rev=1407837&r1=1407836&r2=1407837&view=diff
==============================================================================
--- jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSampleResult.java (original)
+++ jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSampleResult.java Sat Nov 10 15:30:57 2012
@@ -94,7 +94,13 @@ public class HTTPSampleResult extends Sa
      * @return true iif res is an HTTP redirect response
      */
     public boolean isRedirect() {
-        final String[] REDIRECT_CODES = { "301", "302", "303" }; // NOT 304!
+        /*
+         * Don't redirect the following:
+         * 304 = Not Modified
+         * 305 = Use Proxy
+         * 306 = (Unused)
+         */
+        final String[] REDIRECT_CODES = { "301", "302", "303" };
         String code = getResponseCode();
         for (int i = 0; i < REDIRECT_CODES.length; i++) {
             if (REDIRECT_CODES[i].equals(code)) {