<div dir="ltr">Attached is a new diff based on our discussion on irc. <br>This is now handled within the vfps and thus will happen after vcl_backend_response and the inserted object will include the Vary header.<br><br>The caveat with this approach, at least in the current implementation, is that disabling http_gzip_support won't have any effect on existing objects but I can't see this being an issue.<br>
<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Aug 26, 2014 at 1:26 PM, Poul-Henning Kamp <span dir="ltr"><<a href="mailto:phk@phk.freebsd.dk" target="_blank">phk@phk.freebsd.dk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">--------<br>
In message <CAJV_h0bfQstBU=hQ60=<a href="mailto:HDhAyp8AGwVhRx30_OBdQ10Sp4R1ufA@mail.gmail.com">HDhAyp8AGwVhRx30_OBdQ10Sp4R1ufA@mail.gmail.com</a>><br>
<div class="">, Federico Schwindt writes:<br>
<br>
>Does this mean that doing it in vfp_gzip_init() or cache_gzip.c would be<br>
>appropriate or preferred?<br>
<br>
</div>I probably lost context for "it" here :-)<br>
<br>
But if you look in -trunk, you'll see for instance vfp_gzip_init()<br>
doing:<br>
<br>
        if (vfe->vfp->priv2 == VFP_GUNZIP || vfe->vfp->priv2 == VFP_GZIP) {<br>
                http_Unset(vc->http, H_Content_Encoding);<br>
                http_Unset(vc->http, H_Content_Length);<br>
                RFC2616_Weaken_Etag(vc->http);<br>
<div class="HOEnZb"><div class="h5">        }<br>
<br>
<br>
--<br>
Poul-Henning Kamp       | UNIX since Zilog Zeus 3.20<br>
phk@FreeBSD.ORG         | TCP/IP since RFC 956<br>
FreeBSD committer       | BSD since 4.3-tahoe<br>
Never attribute to malice what can adequately be explained by incompetence.<br>
</div></div></blockquote></div><br></div>