[nginx] Reset c->log->action on redirect to a named location.

Maxim Dounin mdounin at mdounin.ru
Sat Apr 27 15:56:46 UTC 2024


details:   http://freenginx.org/hg/nginx/rev/a6649497a762
branches:  
changeset: 9253:a6649497a762
user:      Maxim Dounin <mdounin at mdounin.ru>
date:      Sat Apr 27 18:18:36 2024 +0300
description:
Reset c->log->action on redirect to a named location.

This matches what happens on internal redirects in ngx_http_handler(), and
ensures that stale log actions, such as "sending to client" from the upstream
module, are not used after an error_page redirection to a named location.

diffstat:

 src/http/ngx_http_core_module.c |  2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diffs (12 lines):

diff --git a/src/http/ngx_http_core_module.c b/src/http/ngx_http_core_module.c
--- a/src/http/ngx_http_core_module.c
+++ b/src/http/ngx_http_core_module.c
@@ -2596,6 +2596,8 @@ ngx_http_named_location(ngx_http_request
                            "using location: %V \"%V?%V\"",
                            name, &r->uri, &r->args);
 
+            r->connection->log->action = NULL;
+
             r->internal = 1;
             r->content_handler = NULL;
             r->uri_changed = 0;



More information about the nginx-devel mailing list