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>


Reply via email to