Opened 8 years ago

Closed 8 years ago

Last modified 4 years ago

#970 closed defect (invalid)

[Webdav] : Displayname entity member shouldn't be URL-encoded

Reported by: arnova@… Owned by:
Priority: minor Milestone:
Component: nginx-module Version: 1.2.x
Keywords: webdav Cc:
uname -a: Linux data01 2.6.32-5-amd64 #1 SMP Mon Sep 23 22:14:43 UTC 2013 x86_64 GNU/Linux
nginx -V: nginx version: nginx/1.2.1
TLS SNI support enabled
configure arguments: --prefix=/etc/nginx --conf-path=/etc/nginx/nginx.conf --error-log-path=/var/log/nginx/error.log --http-client-body-temp-path=/var/lib/nginx/body --http-fastcgi-temp-path=/var/lib/nginx/fastcgi --http-log-path=/var/log/nginx/access.log --http-proxy-temp-path=/var/lib/nginx/proxy --http-scgi-temp-path=/var/lib/nginx/scgi --http-uwsgi-temp-path=/var/lib/nginx/uwsgi --lock-path=/var/lock/nginx.lock --pid-path=/var/run/nginx.pid --with-pcre-jit --with-debug --with-http_addition_module --with-http_dav_module --with-http_geoip_module --with-http_gzip_static_module --with-http_image_filter_module --with-http_realip_module --with-http_stub_status_module --with-http_ssl_module --with-http_sub_module --with-http_xslt_module --with-ipv6 --with-sha1=/usr/include/openssl --with-md5=/usr/include/openssl --with-mail --with-mail_ssl_module --add-module=/build/nginx-1.2.1/debian/modules/nginx-auth-pam --add-module=/build/nginx-1.2.1/debian/modules/nginx-echo --add-module=/build/nginx-1.2.1/debian/modules/nginx-upstream-fair --add-module=/build/nginx-1.2.1/debian/modules/nginx-dav-ext-module

Description

Currently nginx's webdav module URL-encodes the value for the "displayname" entity member. According to RFC2518 (e.g. section 13.2) it should contain a "name suitable for representation to the user". They don't mention it should be URL encoded, and I interpret this as a "plain" displayname. Furthermore since it's only for display, and not a real path, why should it be URL-encoded?

Note that all other Webdav-servers I've used until now (Apache, Lighttp, IIS) also do NOT URL encode the displayname, so I believe nginx is at fault here.

Change History (2)

comment:1 by Maxim Dounin, 8 years ago

Resolution: invalid
Status: newclosed

No displayname is returned by nginx at all, as it doesn't support the PROPFIND method. You may want to report this problem elsewhere (likely, to the dav-ext module issue tracker).

comment:2 by stokito@…, 4 years ago

The corresponding issue is reported here https://github.com/arut/nginx-dav-ext-module/issues/44

Note: See TracTickets for help on using tickets.