[PATCH] Cache: check if ETag and Vary didn't change before updating header

Maxim Dounin mdounin at mdounin.ru
Tue Dec 23 01:16:25 UTC 2014


Hello!

On Mon, Dec 22, 2014 at 02:58:43PM -0800, Piotr Sikora wrote:

> Hey Maxim,
> 
> > The check in question is mostly a safeguard to ensure that
> > ngx_file_uniq() check above did it's work properly.
> 
> I'm aware of that.
> 
> > Any specific reason to assume it needs to be extended?
> 
> It just seems inconsistent to check only some of the fields that
> aren't supposed to change (i.e. version, last modified, etc.), but not
> the others (i.e. etag and vary).

Ok, understood.  I don't think this check needs to be extended - as 
of now it checks basic fixed-width fields, and I think these 
checks should be sufficient.

-- 
Maxim Dounin
http://nginx.org/



More information about the nginx-devel mailing list