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