mod_proxy with ProxyErrorOverride On in a reverse-proxy configuration attaches

a body to the 302 response and a wrong Content-Length header.

PR: 22951
Submitted by:	Ermanno Scaglione <scaglione ..at.. starnetone.de>
Reviewed by:	Blaise Tarr <blaise.tarr .. at .. cnet.com>

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@102069 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/CHANGES b/CHANGES
index 2253ef93c237b12b08b2eaeabea483a29b1dde71..690d93425046f0d1a2b378da3569531442d27555 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -2,6 +2,10 @@ Changes with Apache 2.1.0-dev
 
   [Remove entries to the current 2.0 section below, when backported]
 
+  *) mod_proxy with ProxyErrorOverride On in a reverse-proxy configuration attaches
+     a body to the 302 response and a wrong Content-Length header. 
+     PR: 22951 [Ermanno Scaglione scaglione ..at.. starnetone.de]
+
   *) Bring ErrorHeader concept forward from 1.3, so that response
      header fields can be set for return even on errors or external
      redirects.  [Ken Coar]

diff --git a/modules/proxy/proxy_http.c b/modules/proxy/proxy_http.c
index 18bc5a881951d6f973e31d0cdf317e18e2922015..35da0b78089207650e3e58bc60eb9b4a02986d82 100644 (file)
--- a/modules/proxy/proxy_http.c
+++ b/modules/proxy/proxy_http.c
@@ -1020,7 +1020,7 @@ apr_status_t ap_proxy_http_process_response(apr_pool_t * p, request_rec *r,
              * if we are overriding the errors, we can't put the content
              * of the page into the brigade
              */
-            if ( (conf->error_override ==0) || r->status < 400 ) {
+            if ( (conf->error_override ==0) || r->status < 300 ) {
 
                 /* read the body, pass it to the output filters */
                 int finish = FALSE;