[PATCH] For Vary processing, treat an empty header (only whitespace), like a non-existent header[PATCH] For Vary processing, treat an empty header (only whitespace), like a non-existent header

[Poul-Henning Kamp]

--------
In message <549EFB94.6050206 at schokola.de>, Nils Goroll writes:

>Our interpretation of http://tools.ietf.org/html/rfc7234#section-4.1 is that
>removing all whitespace from a header field only containing whitespace
>equals to removing the header.

I do not agree.

It depends on the definition of each header if an empty header is
the same as an absent header, and I really do not want to add a lot
of code to handle that corner-case.

Varnish's Vary processing is by design rather simplistic, we do not
attempt to normalize headers etc.

If you want to wash your headers before vary processing, by all means
do so in VCL.

-- 
Poul-Henning Kamp       | UNIX since Zilog Zeus 3.20
phk at FreeBSD.ORG         | TCP/IP since RFC 956
FreeBSD committer       | BSD since 4.3-tahoe    
Never attribute to malice what can adequately be explained by incompetence.