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/08/08 03:26:37 UTC

svn commit: r230725 - /httpd/httpd/branches/proxy-reqbody-2.0.x/modules/proxy/proxy_http.c

Author: wrowe
Date: Sun Aug  7 18:26:34 2005
New Revision: 230725

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

  Backport the fix to an edge case; it's now much more efficient to 
  entirely skip request body determinations for subrequests, stuff
  in an EOS and we are off to the races, ready to create a body-less
  proxied subrequest.

Modified:
    httpd/httpd/branches/proxy-reqbody-2.0.x/modules/proxy/proxy_http.c

Modified: httpd/httpd/branches/proxy-reqbody-2.0.x/modules/proxy/proxy_http.c
URL: http://svn.apache.org/viewcvs/httpd/httpd/branches/proxy-reqbody-2.0.x/modules/proxy/proxy_http.c?rev=230725&r1=230724&r2=230725&view=diff
==============================================================================
--- httpd/httpd/branches/proxy-reqbody-2.0.x/modules/proxy/proxy_http.c (original)
+++ httpd/httpd/branches/proxy-reqbody-2.0.x/modules/proxy/proxy_http.c Sun Aug  7 18:26:34 2005
@@ -1031,6 +1031,10 @@
             old_te_val = NULL;
             apr_table_unset(r->headers_in, "Transfer-Encoding");
         }
+        rb_method = RB_STREAM_CL;
+        e = apr_bucket_eos_create(input_brigade->bucket_alloc);
+        APR_BRIGADE_INSERT_TAIL(input_brigade, e);
+        goto skip_body;
     }
 
     /* Prefetch MAX_MEM_SPOOL bytes
@@ -1124,6 +1128,9 @@
             rb_method = RB_SPOOL_CL;
         }
     }
+
+/* Yes I hate gotos.  This is the subrequest shortcut */
+skip_body:
 
     switch(rb_method) {
     case RB_STREAM_CHUNKED: