From 4e275cc73a402e7508309a6612c774d21dffc88e Mon Sep 17 00:00:00 2001 From: Maxim Dounin Date: Thu, 26 Jun 2014 02:19:58 +0400 Subject: [PATCH] Upstream: no need to clear r->headers_out.last_modified_time. Clearing of the r->headers_out.last_modified_time field if a response isn't cacheable in ngx_http_upstream_send_response() was introduced in 3b6afa999c2f, the commit to enable not modified filter for cacheable responses. It doesn't make sense though, as at this point header was already sent, and not modified filter was already executed. Therefore, the line was removed to simplify code. --- src/http/ngx_http_upstream.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/http/ngx_http_upstream.c b/src/http/ngx_http_upstream.c index f7ac4a4eb..4a4af7321 100644 --- a/src/http/ngx_http_upstream.c +++ b/src/http/ngx_http_upstream.c @@ -2500,7 +2500,6 @@ ngx_http_upstream_send_response(ngx_http_request_t *r, ngx_http_upstream_t *u) } else { u->cacheable = 0; - r->headers_out.last_modified_time = -1; } }