mod_proxy/4302: It would be useful for response headers added by other modules to be added to the server's response.

[Matt Hoskins <ma...@nipltd.com>]

>Number:         4302
>Category:       mod_proxy
>Synopsis:       It would be useful for response headers added by other modules to be added to the server's response.
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    apache
>State:          open
>Class:          change-request
>Submitter-Id:   apache
>Arrival-Date:   Sat Apr 24 07:10:01 PDT 1999
>Last-Modified:
>Originator:     matt@nipltd.com
>Organization:
apache
>Release:        1.3.6
>Environment:
Debian 2.1 (Linux kernel 2.0.36)
>Description:
The specific case is that I want mod_usertrack's cookies to be sent for
proxied requests as well as normal requests. It's conceivable there may be other
cases where it'd be useful to add headers other than cookies to the proxied
output, but I can't think of any right now :).
>How-To-Repeat:

>Fix:
A simple addition to proxy_http.c to allow this is that after the line which 
reads:
      ap_table_do(ap_proxy_send_hdr_line, &tdo, resp_hdrs, NULL);
you add:
      ap_table_do(ap_proxy_send_hdr_line, &tdo, r->headers_out, NULL);

I haven't investigated fully but I guess this shouldn't cause backward
compatibility problems as where people are using the proxy facility, they won't
be adding additional headers for output to the client.
>Audit-Trail:
>Unformatted:
[In order for any reply to be added to the PR database, ]
[you need to include <ap...@Apache.Org> in the Cc line ]
[and leave the subject line UNCHANGED.  This is not done]
[automatically because of the potential for mail loops. ]
[If you do not include this Cc, your reply may be ig-   ]
[nored unless you are responding to an explicit request ]
[from a developer.                                      ]
[Reply only with text; DO NOT SEND ATTACHMENTS!         ]