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

fix DAV class from 2 to 1,2 (like apache) #46

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

avlubimov
Copy link

@avlubimov avlubimov commented Aug 20, 2019

I have found that most clients require a dav class1 to work

For example, davfs2 or fuse-wdfs check class1 and if they don't find it, they write it.

/sbin/mount.davfs: mounting failed; the server does not support WebDAV

I checked with wireshark that the lua-nginx-module puts the header "dav"="2" while the apache server puts the header "dav"="1,2" and therefore davfs and wdfs work with apache without problems and do not work with nginx.

The proposed patch displays the dav version as well as apache.

@rozhuk-im
Copy link

rozhuk-im commented Mar 12, 2020

add_header DAV '1, 2'; in config solve this.
More hacks: http://netlab.dhis.org/wiki/ru:software:nginx:webdav

@avlubimov
Copy link
Author

add_header DAV '1, 2'; in config solve this.
More hacks: http://netlab.dhis.org/wiki/ru:software:nginx:webdav

Отличная страница, читал ее. На мой взгляд, вполне можно исправить в коде то, что не работает по дефолту.

Кстати, вопрос по этой странице, я так и не понял, зачем в конфиг ставить ограничение

client_max_body_size 50m;

Фактически, это значит, что файлы больше 50 мб загрузить на этот сервер не получится?

@AnrDaemon
Copy link

Not, that means requests with body part larger than 50MB of declared length won't be handled by the server.
Which not necessarily equals the file size.

@rozhuk-im
Copy link

@avlubimov Насколько я понял, всё то что я хакаю в конфиге - это отклонение от стандарта описанного в RFC.
Кажется я даже репортил про слеши на конце в nginx мне сказали что у них по стандарту и дали ссылку.

client_max_body_size - помнится это ограничение не на размер файла:

Sets the maximum allowed size of the client request body

add_header Accept-Ranges 'bytes'; - это то что я ещё добавил много где после.

mid1221213 added a commit to mid1221213/nginx-dav-ext-module that referenced this pull request Nov 5, 2021
mid1221213 added a commit to mid1221213/nginx-dav-ext-module that referenced this pull request Nov 5, 2021
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

Successfully merging this pull request may close these issues.

3 participants