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


Thanks,
Zev