]> granicus.if.org Git - apache/commit
Merge r1808746, r1809028 from trunk:
authorYann Ylavic <ylavic@apache.org>
Fri, 13 Oct 2017 08:42:57 +0000 (08:42 +0000)
committerYann Ylavic <ylavic@apache.org>
Fri, 13 Oct 2017 08:42:57 +0000 (08:42 +0000)
commit90c6edccd9473213c2fd4826216891989234f47e
tree33399801f58b1936ab0ec2a55174a20bfb89f35f
parentecc0f862c1042a471bfe84c3aaba1c6d8ba26f94
Merge r1808746, r1809028 from trunk:

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.

mod_rewrite,core: avoid Vary:Host (part 2)

This is a follow up of r1808746 after a chat
with Yann on dev@:

- the HTTP:Host variable suffers from the same problem
- the strcasecmp should be used to allow case-sensitive
  comparisons.
- in mod_rewrite is less cumbersome and more clean to just
  make the Host header check in lookup_header, so it will
  be automatically picked up by every part of the code
  that uses it. It shouldn't be a relevant overhead for
  mod_rewrite.

Submitted by: elukey
Reviewed by: elukey, ylavic, wrowe

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