http://issues.apache.org/bugzilla/show_bug.cgi?id=39245

In short, when ProxyErrorOverride is on httpd will try to replace the
body/headers of a 302 response -- which unnecessarily causes cookies
to be dropped.

There's a simple patch attached to the report which tells httpd to
keep its hands off ap_is_HTTPD_REDIRECT e.g. 3xx.  Given the
description of ProxyErrorOverride I wouldn't think expectations would
be that 3xx responses would be overridden.

Can any proxy gurus reconsider this bug status?

--- httpd-2.2.0-orig/modules/proxy/mod_proxy_http.c     2005-11-10
16:20:05.000000000 +0100
+++ httpd-2.2.0/modules/proxy/mod_proxy_http.c  2006-03-31
18:02:26.000000000 +0200
@@ -1447,7 +1447,7 @@
             * if we are overriding the errors, we can't put the content
             * of the page into the brigade
             */
-            if (conf->error_override == 0 || ap_is_HTTP_SUCCESS(r->status)) {
+            if (conf->error_override == 0 ||
ap_is_HTTP_SUCCESS(r->status) || ap_is_HTTP_REDIRECT(r->status)) {
                /* read the body, pass it to the output filters */
                apr_read_type_e mode = APR_NONBLOCK_READ;
                int finish = FALSE;
@@ -1549,7 +1549,7 @@

    if (conf->error_override) {
        /* the code above this checks for 'OK' which is what the hook
expects */
-        if (ap_is_HTTP_SUCCESS(r->status))
+        if (ap_is_HTTP_SUCCESS(r->status) || ap_is_HTTP_REDIRECT(r->status))
            return OK;
        else {
            /* clear r->status for override error, otherwise ErrorDocument


--
Eric Covener
[EMAIL PROTECTED]

Reply via email to