You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jmeter-dev@jakarta.apache.org by se...@apache.org on 2007/10/20 17:58:40 UTC

svn commit: r586751 - in /jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler: HTTPSampler.java HTTPSampler2.java

Author: sebb
Date: Sat Oct 20 08:58:39 2007
New Revision: 586751

URL: http://svn.apache.org/viewvc?rev=586751&view=rev
Log:
Transfer-Encoding header now handled by Proxy

Modified:
    jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSampler.java
    jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSampler2.java

Modified: jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSampler.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSampler.java?rev=586751&r1=586750&r2=586751&view=diff
==============================================================================
--- jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSampler.java (original)
+++ jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSampler.java Sat Oct 20 08:58:39 2007
@@ -279,14 +279,10 @@
 
         String hfk;
 		for (int i = 1; (hfk=conn.getHeaderFieldKey(i)) != null; i++) {
-            // TODO: Handle this in Proxy instead?
-			// As this has already been handled, it is currently removed to avoid confusing the JMeter Proxy
-            if (!TRANSFER_ENCODING.equalsIgnoreCase(hfk)) {
-                headerBuf.append(hfk);
-                headerBuf.append(": "); // $NON-NLS-1$
-                headerBuf.append(conn.getHeaderField(i));
-                headerBuf.append("\n"); // $NON-NLS-1$
-            }
+            headerBuf.append(hfk);
+            headerBuf.append(": "); // $NON-NLS-1$
+            headerBuf.append(conn.getHeaderField(i));
+            headerBuf.append("\n"); // $NON-NLS-1$
 		}
 		return headerBuf.toString();
 	}

Modified: jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSampler2.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSampler2.java?rev=586751&r1=586750&r2=586751&view=diff
==============================================================================
--- jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSampler2.java (original)
+++ jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSampler2.java Sat Oct 20 08:58:39 2007
@@ -604,15 +604,10 @@
 
 		for (int i = 0; i < rh.length; i++) {
 			String key = rh[i].getName();
-            // TODO: Handle this in Proxy instead?
-			// As this has already been handled, it is currently removed to avoid confusing the JMeter Proxy
-			if (!key.equalsIgnoreCase(TRANSFER_ENCODING))
-			{
-				headerBuf.append(key);
-				headerBuf.append(": "); // $NON-NLS-1$
-				headerBuf.append(rh[i].getValue());
-				headerBuf.append("\n"); // $NON-NLS-1$
-			}
+			headerBuf.append(key);
+			headerBuf.append(": "); // $NON-NLS-1$
+			headerBuf.append(rh[i].getValue());
+			headerBuf.append("\n"); // $NON-NLS-1$
 		}
 		return headerBuf.toString();
 	}



---------------------------------------------------------------------
To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org