Mercurial > hg > nginx
diff src/http/ngx_http_upstream.c @ 6242:0e3a45ec2a3a
Upstream: fixed cache send error handling.
The value of NGX_ERROR, returned from filter handlers, was treated as a generic
upstream error and changed to NGX_HTTP_INTERNAL_SERVER_ERROR before calling
ngx_http_finalize_request(). This resulted in "header already sent" alert
if header was already sent in filter handlers.
The problem appeared in 54e9b83d00f0 (1.7.5).
author | Roman Arutyunyan <arut@nginx.com> |
---|---|
date | Thu, 03 Sep 2015 15:09:21 +0300 |
parents | 8b6fa4842133 |
children | 257b51c37c5a |
line wrap: on
line diff
--- a/src/http/ngx_http_upstream.c Wed Sep 02 19:45:40 2015 +0300 +++ b/src/http/ngx_http_upstream.c Thu Sep 03 15:09:21 2015 +0300 @@ -534,15 +534,24 @@ r->write_event_handler = ngx_http_request_empty_handler; - if (rc == NGX_DONE) { - return; - } - if (rc == NGX_ERROR) { ngx_http_finalize_request(r, NGX_HTTP_INTERNAL_SERVER_ERROR); return; } + if (rc == NGX_OK) { + rc = ngx_http_upstream_cache_send(r, u); + + if (rc == NGX_DONE) { + return; + } + + if (rc == NGX_HTTP_UPSTREAM_INVALID_HEADER) { + rc = NGX_DECLINED; + r->cached = 0; + } + } + if (rc != NGX_DECLINED) { ngx_http_finalize_request(r, rc); return; @@ -837,13 +846,7 @@ case NGX_OK: - rc = ngx_http_upstream_cache_send(r, u); - - if (rc != NGX_HTTP_UPSTREAM_INVALID_HEADER) { - return rc; - } - - break; + return NGX_OK; case NGX_HTTP_CACHE_STALE: