]> git.kaiwu.me - nginx.git/commit
Upstream: fixed X-Accel-Expires/Cache-Control/Expires handling.
authorMaxim Dounin <mdounin@mdounin.ru>
Mon, 6 Jun 2022 21:07:12 +0000 (00:07 +0300)
committerMaxim Dounin <mdounin@mdounin.ru>
Mon, 6 Jun 2022 21:07:12 +0000 (00:07 +0300)
commit8df3ad13c5bb10360221f25ca45ae03dd09e1c3b
treebb56dd34543c41b8cc92ebf52ff4b20e33273b1c
parentbfc5b35827903a3c543b58e4562db8b62021c164
Upstream: fixed X-Accel-Expires/Cache-Control/Expires handling.

Previously, if caching was disabled due to Expires in the past, nginx
failed to cache the response even if it was cacheable as per subsequently
parsed Cache-Control header (ticket #964).

Similarly, if caching was disabled due to Expires in the past,
"Cache-Control: no-cache" or "Cache-Control: max-age=0", caching was not
used if it was cacheable as per subsequently parsed X-Accel-Expires header.

Fix is to avoid disabling caching immediately after parsing Expires in
the past or Cache-Control, but rather set flags which are later checked by
ngx_http_upstream_process_headers() (and cleared by "Cache-Control: max-age"
and X-Accel-Expires).

Additionally, now X-Accel-Expires does not prevent parsing of cache control
extensions, notably stale-while-revalidate and stale-if-error.  This
ensures that order of the X-Accel-Expires and Cache-Control headers is not
important.

Prodded by Vadim Fedorenko and Yugo Horie.
src/http/ngx_http_upstream.c
src/http/ngx_http_upstream.h