]> granicus.if.org Git - apache/commit
mod_rewrite/core: avoid the 'Vary: Host' header
authorLuca Toscano <elukey@apache.org>
Mon, 18 Sep 2017 17:08:54 +0000 (17:08 +0000)
committerLuca Toscano <elukey@apache.org>
Mon, 18 Sep 2017 17:08:54 +0000 (17:08 +0000)
commitbc0f112313efded6fe63b39442a8fbc4470678c5
tree9db072b280961153de13f63da97e6a6c442b0c7b
parentf77ced89e76d5f3b15f14b6cb7ee13e5c785e0bf
mod_rewrite/core: avoid the 'Vary: Host' header

In PR 58231 is was brought up that httpd adds the
Vary: Host header whenever a condition is set to true
in mod_rewrite or in an <If> block.

The https://tools.ietf.org/html/rfc7231#section-7.1.4
section seems to disallow this use case:

"The "Vary" header field in a response describes "
"what parts of a request message, "
"aside from the method, Host header field, [...]"

I had a chat with the folks in #traffic-server and
they don't see much point in having a Vary: Host header,
plus it was reported that Varnish doesn't like it very
much (namely it does not cache the response when
it sees the header, links of the report in the PR).

I don't see much value in this behavior of httpd so
I am inclined to remove this response header value,
but I'd be glad to get a more experienced opinion.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1808746 13f79535-47bb-0310-9956-ffa450edef68
CHANGES
modules/mappers/mod_rewrite.c
server/util_expr_eval.c