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

Piotr Sikora piotr at cloudflare.com
Mon Dec 22 22:58:43 UTC 2014


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).

Best regards,
Piotr Sikora



More information about the nginx-devel mailing list