On Tue, Sep 15, 2009 at 06:33:38PM +0900, Zev Blut wrote:

> Hello,
>
>
> On 09/15/2009 05:23 PM, Igor Sysoev wrote:
> >On Tue, Sep 15, 2009 at 04:20:48PM +0900, Zev Blut wrote:
>
> >>I have attached a debug log from 0.6.39 that has the Vary when it should
> >>not. If you would like I can also attach a debug log from 0.6.32 that
> >>does not include the Vary (which is what I need).
> >
> >This bug was fixed in 0.7.9.
> >The attached patch fixes it in 0.6.x, if you prefer to stay on this
> >version.
>
> Thank you for the patch!
> Unfortunately, it nor does the latest 0.7.62 seem to fix it.
> I am still getting the Vary: Accept-Encoding header for non gzip stuff.
> In both this patch and with 0.7.62
>
> wget -S -O /dev/null http://localhost:8989/images/sprites/badge.png
>
> HTTP/1.1 200 OK
> Server: nginx/0.6.39
> Date: Tue, 15 Sep 2009 09:15:02 GMT
> Content-Type: image/png
> Content-Length: 20031
> Last-Modified: Tue, 08 Sep 2009 07:50:52 GMT
> Connection: keep-alive
> Vary: Accept-Encoding
> Expires: Wed, 16 Sep 2009 09:15:02 GMT
> Cache-Control: max-age=86400
> Accept-Ranges: bytes
>
> --------
>
> HTTP/1.1 200 OK
> Server: nginx/0.7.62
> Date: Tue, 15 Sep 2009 09:28:02 GMT
> Content-Type: image/png
> Content-Length: 20031
> Last-Modified: Tue, 08 Sep 2009 07:50:52 GMT
> Connection: keep-alive
> Vary: Accept-Encoding
> Expires: Wed, 16 Sep 2009 09:28:02 GMT
> Cache-Control: max-age=86400
> Accept-Ranges: bytes
>
>
> --- HERE is a good response example ---
>
> HTTP/1.1 200 OK
> Server: nginx/0.6.32
> Date: Tue, 15 Sep 2009 09:32:50 GMT
> Content-Type: image/png
> Content-Length: 20031
> Last-Modified: Tue, 08 Sep 2009 07:50:52 GMT
> Connection: keep-alive
> Expires: Wed, 16 Sep 2009 09:32:50 GMT
> Cache-Control: max-age=86400
> Accept-Ranges: bytes

I could not reproduce these responses on 0.8.15, 0.7.62 and patched 0.6.39.
Are you sure that you have no "image/png" in gzip_types ?


--
Igor Sysoev
http://sysoev.ru/en/