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 Maxim Dounin, 11 years ago

Status: newaccepted

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.

comment:2 by Maxim Dounin <mdounin@…>, 11 years ago

In 375055da60fae22c8dfa6b385e1f72739fa0b9b6/nginx_org:

Improved wording of merge_slashes etc. (ticket #448).

comment:3 by Maxim Dounin, 11 years ago

Component: nginx-coreother
Resolution: fixed
Status: acceptedclosed

Documentation fix committed, thanks.

Note: See TracTickets for help on using tickets.