[PATCH] Cache: send conditional requests only for cached 200/206 responses

Maxim Dounin mdounin at mdounin.ru
Wed Nov 26 14:55:56 UTC 2014


Hello!

On Tue, Nov 25, 2014 at 05:40:11PM -0800, Piotr Sikora wrote:

[...]

>          if (valid) {
> +            r->cache->status = u->headers_in.status_n;
>              r->cache->last_modified = u->headers_in.last_modified_time;
>              r->cache->date = now;
>              r->cache->body_start = (u_short) (u->buffer.pos - u->buffer.start);

I think we should consider not saving last_modified and etag 
instead.

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



More information about the nginx-devel mailing list