Author: rjung
Date: Wed Jul  1 09:06:22 2009
New Revision: 790080

URL: http://svn.apache.org/viewvc?rev=790080&view=rev
Log:
Mark answered the questions, so now vote +1.

Modified:
    tomcat/current/tc5.5.x/STATUS.txt

Modified: tomcat/current/tc5.5.x/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/current/tc5.5.x/STATUS.txt?rev=790080&r1=790079&r2=790080&view=diff
==============================================================================
--- tomcat/current/tc5.5.x/STATUS.txt (original)
+++ tomcat/current/tc5.5.x/STATUS.txt Wed Jul  1 09:06:22 2009
@@ -75,16 +75,8 @@
   Handle chunked POSTs. If maxPostSize is exceeded, a 500 results
   http://svn.apache.org/viewvc?rev=785381&view=rev
   http://svn.apache.org/viewvc?rev=786487&view=rev
-  +1: markt
+  +1: markt, rjung
   -1: 
-  rjung: +1 in principle, but a few questions/suggestions:
-      - Do we need to dechunk the read request body? I didn't see were this 
happens.
-      - Do chunked requests actually set a content-length? If not, we have len 
== -1
-        and will not enter the try/catch block. Maybe only a question of 
correct backport
-        from trunk.
-      - I think instead of: len == actualLen => non-chunked, else check 
transfer-encoding
-        it should be the other waqy round. If transfer-encoding is chunked, 
handle as chunked,
-        else traditional.
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=38352
   JSPs should have read/write access to the context's temp dir



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

Reply via email to