Opened 11 years ago
Closed 11 years ago
#448 closed defect (fixed)
merge_slashes off doesn't work inside server context
Reported by: | Jason Barnabe | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | other | Version: | |
Keywords: | Cc: | ||
uname -a: | Linux jason-thinkpad 3.11.0-13-generic #20-Ubuntu SMP Wed Oct 23 07:38:26 UTC 2013 x86_64 x86_64 x86_64 GNU/Linux | ||
nginx -V: |
nginx version: nginx/1.4.2
built by gcc 4.8.1 (Ubuntu/Linaro 4.8.1-10ubuntu8) TLS SNI support enabled configure arguments: --prefix=/opt/nginx --with-http_ssl_module --with-http_gzip_static_module --with-http_stub_status_module --with-cc-opt=-Wno-error --with-pcre=/tmp/passenger.1kxt1z/pcre-8.32 --add-module=/home/jason/.rvm/gems/ruby-2.0.0-p247/gems/passenger-4.0.20/ext/nginx |
Description
When I put merge_slashes off in the http context, it works. When I put it in my server context, it doesn't. This may or may not be expected, the documentation has this ambiguous statement:
"A directive can be specified on the server level in the default server. In this case, its value will apply to all virtual servers listening on the same address and port."
Does this mean it only works in the default server? Or does it mean that it works in all servers, but putting it in the default server has this special meaning? If it's the former, may I suggest this updated wording:
"This directive may be specified on the server level only for default servers. In this case, its value will apply to all virtual servers listening on the same address and port. Including this directive on non-default servers has no effect."
If it's the latter, then consider this a bug.
Change History (3)
comment:1 by , 11 years ago
Status: | new → accepted |
---|
comment:3 by , 11 years ago
Component: | nginx-core → other |
---|---|
Resolution: | → fixed |
Status: | accepted → closed |
Documentation fix committed, thanks.
When specified at server{} level, it only works if a server is a default one. Improving wording for this and other similar directives makes sense.