<div dir="ltr">This cannot happen before v_b_r as the gzip'ness of the object could be changed in there.<br>It needs to happen after, or before and when changing do_gzip and do_gunzip.<br><br></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Tue, Aug 26, 2014 at 11:46 AM, Nils Goroll <span dir="ltr"><<a href="mailto:slink@schokola.de" target="_blank">slink@schokola.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
As discussed on IRC: I do very much appreciate that you've but this back onto<br>
the agenda, this is important and Varnish needs to get the default Vary right.<br>
<br>
But I think this needs to live in fetch (before calling v_b_r): The Vary header<br>
does not depend upon the request, so it does not need to live in deliver. Doing<br>
things at delivery time is more costly than at fetch time (because we probably<br>
cache the result).<br>
<br>
When backends handle gzip, we also end up with a cache-object with Vary: A-E (at<br>
least for sane backends), so we should get the same result when Varnish does the<br>
compression.<br>
<br>
vry_cmp already skips Accept-Encoding, so we should be fine at lookup time.<br>
<span class="HOEnZb"><font color="#888888"><br>
Nils<br>
</font></span></blockquote></div><br></div>