Mercurial > hg > nginx
comparison src/http/ngx_http_core_module.c @ 4375:f31146f282fd
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.
author | Maxim Dounin <mdounin@mdounin.ru> |
---|---|
date | Mon, 19 Dec 2011 14:11:48 +0000 |
parents | b278e6e514fd |
children | d620f497c50f |
comparison
equal
deleted
inserted
replaced
4374:ae376a94de11 | 4375:f31146f282fd |
---|---|
2504 #if (NGX_HTTP_CACHE) | 2504 #if (NGX_HTTP_CACHE) |
2505 r->cache = NULL; | 2505 r->cache = NULL; |
2506 #endif | 2506 #endif |
2507 | 2507 |
2508 r->internal = 1; | 2508 r->internal = 1; |
2509 r->valid_unparsed_uri = 0; | |
2509 r->add_uri_to_alias = 0; | 2510 r->add_uri_to_alias = 0; |
2510 r->main->count++; | 2511 r->main->count++; |
2511 | 2512 |
2512 ngx_http_handler(r); | 2513 ngx_http_handler(r); |
2513 | 2514 |