Skip to content

Commit

Permalink
Upstream: no need to clear r->headers_out.last_modified_time.
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
mdounin committed Jun 25, 2014
1 parent 05dfc05 commit 4e275cc
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion src/http/ngx_http_upstream.c
Original file line number Diff line number Diff line change
Expand Up @@ -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;
}
}

Expand Down

0 comments on commit 4e275cc

Please sign in to comment.