You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by wr...@apache.org on 2005/07/19 23:59:18 UTC

svn commit: r219782 - /httpd/httpd/branches/2.0.x/STATUS

Author: wrowe
Date: Tue Jul 19 14:59:17 2005
New Revision: 219782

URL: http://svn.apache.org/viewcvs?rev=219782&view=rev
Log:

  Vetoed the first attempt at a 2.1-dev mod_proxy_http request body
  backport - so note this as a showstopper rather than reverting the code.

  Also note the RFC bug that the -trace.patch corrects.

Modified:
    httpd/httpd/branches/2.0.x/STATUS

Modified: httpd/httpd/branches/2.0.x/STATUS
URL: http://svn.apache.org/viewcvs/httpd/httpd/branches/2.0.x/STATUS?rev=219782&r1=219781&r2=219782&view=diff
==============================================================================
--- httpd/httpd/branches/2.0.x/STATUS (original)
+++ httpd/httpd/branches/2.0.x/STATUS Tue Jul 19 14:59:17 2005
@@ -167,6 +167,26 @@
                  the issues of correctly sending the body and choosing the
                  transport flavor.
 
+    * http://svn.apache.org/viewcvs?rev=171205&view=rev [committed]
+      backport this fix from 2.1-dev:
+      proxy HTTP: Rework the handling of request bodies to handle
+      chunked input and input filters which modify content length, and
+      avoid spooling arbitrary-sized request bodies in memory.
+      PR: 15859
+
+        +1 trawick, jerenkrantz, jim
+        -1 wrowe
+           This patch needs to be reverted or ammended by the patch above;
+           this resulting code is more complex, yet equally faulty in it's
+           C-L/T-E elections for a number of specific cases.  No opinion
+           between the choice of reverting and re-backporting, or simply
+           patching-the-patch.
+
+    * TRACE must not have a request body per RFC2616; see the -trace.patch
+      below for one of two alternatives.  The other alternative; simply
+      hack mod_proxy.c to reject TRACE when a body is seen, again see that
+      -trace.patch for an illustration.
+
 
 PATCHES ACCEPTED TO BACKPORT FROM TRUNK:
   [ please append new backports at the end of this list not the top. ]