jmeter-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From pmoua...@apache.org
Subject svn commit: r1299630 - in /jmeter/trunk: src/protocol/http/org/apache/jmeter/protocol/http/proxy/Proxy.java xdocs/changes.xml
Date Mon, 12 Mar 2012 11:24:38 GMT
Author: pmouawad
Date: Mon Mar 12 11:24:38 2012
New Revision: 1299630

URL: http://svn.apache.org/viewvc?rev=1299630&view=rev
Log:
Bug 52885 - Proxy : Recording issues with HTTPS, cookies starting with secure are partly truncated

Modified:
    jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/Proxy.java
    jmeter/trunk/xdocs/changes.xml

Modified: jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/Proxy.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/Proxy.java?rev=1299630&r1=1299629&r2=1299630&view=diff
==============================================================================
--- jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/Proxy.java (original)
+++ jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/Proxy.java Mon Mar
12 11:24:38 2012
@@ -496,7 +496,7 @@ public class Proxy extends Thread {
                     headerLines[i]=headerLines[i].replaceFirst(parts[1].substring(0,HTTPS_PREFIX.length()),
"http://");
                     continue;
                 }
-                if (forcedHTTPS && HTTPConstants.HEADER_COOKIE.equalsIgnoreCase(parts[0])
|| HTTPConstants.HEADER_SET_COOKIE.equalsIgnoreCase(parts[0]))
+                if (forcedHTTPS && (HTTPConstants.HEADER_COOKIE.equalsIgnoreCase(parts[0])
|| HTTPConstants.HEADER_SET_COOKIE.equalsIgnoreCase(parts[0])))
                 {
                     headerLines[i]=headerLines[i].replaceAll(" secure", "").trim(); //in
forced https cookies need to be unsecured...
                 }

Modified: jmeter/trunk/xdocs/changes.xml
URL: http://svn.apache.org/viewvc/jmeter/trunk/xdocs/changes.xml?rev=1299630&r1=1299629&r2=1299630&view=diff
==============================================================================
--- jmeter/trunk/xdocs/changes.xml (original)
+++ jmeter/trunk/xdocs/changes.xml Mon Mar 12 11:24:38 2012
@@ -74,6 +74,7 @@ When doing replacement of User Defined V
 <li>Bug 52781 - Content-Disposition header garbled even if browser compatible headers
is checked (HC4) </li>
 <li>Bug 52796 - MonitorHandler fails to clear variables when starting a new parse</li>
 <li>Bug 52871 - Multiple Certificates not working with HTTP Client 4</li>
+<li>Bug 52885 - Proxy : Recording issues with HTTPS, cookies starting with secure are
partly truncated</li>
 </ul>
 
 <h3>Other Samplers</h3>



Mime
View raw message