From 7fa89c744afd1f05f4405d9a60b8c63a65ea56cc Mon Sep 17 00:00:00 2001 From: Maxim Dounin Date: Fri, 28 Nov 2014 16:57:50 +0300 Subject: [PATCH] Fixed post_action to not trigger "header already sent" alert. The alert was introduced in 03ff14058272 (1.5.4), and was triggered on each post_action invocation. There is no real need to call header filters in case of post_action, so return NGX_OK from ngx_http_send_header() if r->post_action is set. --- src/http/ngx_http_core_module.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/http/ngx_http_core_module.c b/src/http/ngx_http_core_module.c index 3b36f4a19..d8ff53bfe 100644 --- a/src/http/ngx_http_core_module.c +++ b/src/http/ngx_http_core_module.c @@ -1973,6 +1973,10 @@ ngx_http_send_response(ngx_http_request_t *r, ngx_uint_t status, ngx_int_t ngx_http_send_header(ngx_http_request_t *r) { + if (r->post_action) { + return NGX_OK; + } + if (r->header_sent) { ngx_log_error(NGX_LOG_ALERT, r->connection->log, 0, "header already sent");