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 2012/11/08 13:03:35 UTC

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

Author: pmouawad
Date: Thu Nov  8 12:03:34 2012
New Revision: 1407028

URL: http://svn.apache.org/viewvc?rev=1407028&view=rev
Log:
Bug 54119 - HTTP 307 response is not redirected
Docs

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=1407028&r1=1407027&r2=1407028&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 Thu Nov  8 12:03:34 2012
@@ -101,6 +101,11 @@ public class HTTPSampleResult extends Sa
                 return true;
             }
         }
+        // http://www.w3.org/Protocols/rfc2616/rfc2616-sec10.html
+        // If the 307 status code is received in response to a request other than GET or HEAD, 
+        // the user agent MUST NOT automatically redirect the request unless it can be confirmed by the user,
+        // since this might change the conditions under which the request was issued.
+        // See Bug 54119
         if ("307".equals(code) && 
                 ("GET".equals(getHTTPMethod()) || "HEAD".equals(getHTTPMethod()))) {
             return true;