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;