]> git.kaiwu.me - nginx.git/commit
Added clearing of r->valid_unparsed_uri on internal redirects.
authorMaxim Dounin <mdounin@mdounin.ru>
Mon, 19 Dec 2011 14:11:48 +0000 (14:11 +0000)
committerMaxim Dounin <mdounin@mdounin.ru>
Mon, 19 Dec 2011 14:11:48 +0000 (14:11 +0000)
commitd79f4523f8c3b156be4f80fa1f1615217e30bbfa
tree6036902ded76ac3e0fc9006c904a66cbe1f39dd1
parentc5968a998c9b5cb0bec0b7d914832e43357d41cc
Added clearing of r->valid_unparsed_uri on internal redirects.

This resolves issue with try_files (see ticket #70), configuration like

   location / { try_files $uri /index.php; }
   location /index.php { proxy_pass http://backend; }

caused nginx to use original request uri in a request to a backend.

Historically, not clearing of the r->valid_unparsed_uri on internal redirect
was a feature: it allowed to pass the same request to (another) upstream
server via error_page redirection.  Since then named locations appeared
though, and it's time to start resetting r->valid_unparsed_uri on internal
redirects.  Configurations still using this feature should be converted
to use named locations instead.

Patch by Lanshun Zhou.
src/http/ngx_http_core_module.c
src/http/ngx_http_upstream.c