Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PROPFIND returns HTTP/0.0 in status lines #63

Open
rfc2822 opened this issue Sep 30, 2021 · 0 comments
Open

PROPFIND returns HTTP/0.0 in status lines #63

rfc2822 opened this issue Sep 30, 2021 · 0 comments

Comments

@rfc2822
Copy link

rfc2822 commented Sep 30, 2021

Doing curl -vX PROPFIND -u .... https://www.example.com/ returns

…
…
<D:response>
<D:href>/Detlef.jpg</D:href>
<D:propstat>
<D:prop>…</D:prop>
<D:status>HTTP/0.0 200 OK</D:status>             ← HTTP/0.0
</D:propstat>
</D:response>

According to RFC 4918 14.28, D:status holds a single HTTP status-line as defined in RFC 2616 6.1. Because there is not HTTP/0.0 and the examples in RFC 4918 use HTTP/1.1, I suggest that nginx-dav-ext also uses HTTP/1.1.

Current behavior causes problems with the current dav4jvm library which uses okhttp3 StatusLine, which expects the status line to start with HTTP/1..


Version info:

nginx version: nginx/1.18.0 (Ubuntu)
built with OpenSSL 1.1.1f  31 Mar 2020
TLS SNI support enabled
configure arguments: --with-cc-opt='-g -O2 -fdebug-prefix-map=/build/nginx-KTLRnK/nginx-1.18.0=. -fstack-protector-strong -Wformat -Werror=format-security -fPIC -Wdate-time -D_FORTIFY_SOURCE=2' --with-ld-opt='-Wl,-Bsymbolic-functions -Wl,-z,relro -Wl,-z,now -fPIC' --prefix=/usr/share/nginx --conf-path=/etc/nginx/nginx.conf --http-log-path=/var/log/nginx/access.log --error-log-path=/var/log/nginx/error.log --lock-path=/var/lock/nginx.lock --pid-path=/run/nginx.pid --modules-path=/usr/lib/nginx/modules --http-client-body-temp-path=/var/lib/nginx/body --http-fastcgi-temp-path=/var/lib/nginx/fastcgi --http-proxy-temp-path=/var/lib/nginx/proxy --http-scgi-temp-path=/var/lib/nginx/scgi --http-uwsgi-temp-path=/var/lib/nginx/uwsgi --with-debug --with-compat --with-pcre-jit --with-http_ssl_module --with-http_stub_status_module --with-http_realip_module --with-http_auth_request_module --with-http_v2_module --with-http_dav_module --with-http_slice_module --with-threads --with-http_gzip_static_module --without-http_browser_module --without-http_geo_module --without-http_limit_req_module --without-http_limit_conn_module --without-http_memcached_module --without-http_referer_module --without-http_split_clients_module --without-http_userid_module --add-dynamic-module=/build/nginx-KTLRnK/nginx-1.18.0/debian/modules/http-echo

dav-ext:

ii  libnginx-mod-http-dav-ext              1.18.0-0ubuntu1.2                  amd64        WebDAV missing commands support for Nginx
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant