You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2010/11/10 12:45:37 UTC

svn commit: r1033419 - /tomcat/tc6.0.x/trunk/STATUS.txt

Author: markt
Date: Wed Nov 10 11:45:36 2010
New Revision: 1033419

URL: http://svn.apache.org/viewvc?rev=1033419&view=rev
Log:
Additional patch to address kkolinko's concerns
Remove rjung's vote since the patch is significantly different

Modified:
    tomcat/tc6.0.x/trunk/STATUS.txt

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=1033419&r1=1033418&r2=1033419&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Wed Nov 10 11:45:36 2010
@@ -91,20 +91,10 @@ PATCHES PROPOSED TO BACKPORT:
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49860
   Add support for trailing headers.
   http://svn.apache.org/viewvc?rev=1003461&view=rev
+  http://svn.apache.org/viewvc?rev=1033415&view=rev
   Note: Don't change return type for parseEndChunk(), just return true.
-  +1: markt, rjung
-  -1: kkolinko:
-    Regarding implementation of ChunkedInputFilter.parseHeader():
-    One cannot remember old value of 'pos' (e.g. 'int start = pos')
-    outside of a loop that calls readBytes(). The remembered positions
-    are not valid after the readBytes() call.
-    markt: This is pretty much an exact copy of
-           InternalInputBuffer.parseHeader() which has been working for years.
-           Could you elaborate on where you see the issue. A test case that
-           demonstrates it would be even better.
-    kkolinko: I have replied in Re:r1031024
-      Here is a testcase: https://issues.apache.org/bugzilla/attachment.cgi?id=26265&action=edit
-
+  +1: markt
+  -1:
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=50026
   Force DefaultServlet to serve all resources relative to context root



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