#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:
Sensitive: no
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 (1)

comment:1 Changed 20 months ago by mdounin

  • Resolution set to invalid
  • Status changed from new to closed

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).

Note: See TracTickets for help on using tickets.