From 5209db57ee550dd5644e9165efe3423e69fb406e Mon Sep 17 00:00:00 2001 From: Ian Holsman Date: Mon, 15 Dec 2003 23:04:40 +0000 Subject: [PATCH] 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 Reviewed by: Blaise Tarr git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@102069 13f79535-47bb-0310-9956-ffa450edef68 --- CHANGES | 4 ++++ modules/proxy/proxy_http.c | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/CHANGES b/CHANGES index 2253ef93c2..690d934250 100644 --- 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 18bc5a8819..35da0b7808 100644 --- 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; -- 2.40.0