]> granicus.if.org Git - apache/commit
Merge r1811744 from trunk:
authorJim Jagielski <jim@apache.org>
Mon, 13 Nov 2017 13:31:33 +0000 (13:31 +0000)
committerJim Jagielski <jim@apache.org>
Mon, 13 Nov 2017 13:31:33 +0000 (13:31 +0000)
commit6557d718a6519c9940c8e2785600ab9f17c0dcae
tree0a36e778985a60d8a196fc87a000bc8f681d896f
parent58dc244655f341d682b7b281ee6d4c79fde0918d
Merge r1811744 from trunk:

core, mod_rewrite: introduce the 'redirect-keeps-vary' note
                   to allow proper Vary header insertion when
                   dealing with a RewriteRule in a directory
                   context.

This change is an attempt to fix a long standing problem,
brought up while working on PR 58231. Our documentation clearly
states the following:

"If a HTTP header is used in a condition this header is added
to the Vary header of the response in case the condition
evaluates to true for the request."

This is currently not true for RewriteCond/Rules working in
a directory context, since when an internal redirect happens
all the outstanding response headers get dropped.

There might be a better solution so I am looking forward to
hear more opinions and comments. My goal for a delicate change
like this one would be to affect the least amount of configurations
possible, without triggering unwanted side effects.

If the solution is good for everybody tests will be written
in the suite asap.

Submitted by: elukey
Reviewed by: elukey, icing, ylavic

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.4.x@1815100 13f79535-47bb-0310-9956-ffa450edef68
CHANGES
STATUS
modules/http/http_request.c
modules/mappers/mod_rewrite.c