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

Ensure proper collection path when creating directory #259

Merged
merged 2 commits into from
May 9, 2021

Conversation

jstastny
Copy link
Contributor

@jstastny jstastny commented May 6, 2021

According to the specification (http://www.webdav.org/specs/rfc4918.html#rfc.section.5.2) the collection URI contains the trailing slash. While the specification allows that some implementations may treat the URI without the trailing slash as if it had it, some implementations (nginx namely) is strict about this and only supports the URIs with the trailing slash.

This ensures the URI has the trailing slash before sending it via MKCOL.

jstastny added 2 commits May 6, 2021 15:53
According to the specification (http://www.webdav.org/specs/rfc4918.html#rfc.section.5.2) the collection URI contains the trailing slash. While the specification allows that some implementations may treat the URI without the trailing slash as if it had it, some implementations (nginx namely) is strict about this and only supports the URIs with the trailing slash.

This ensures the URI has the trailing slash before sending it via `MKCOL`.
@perry-mitchell
Copy link
Owner

Great! :)

@perry-mitchell perry-mitchell merged commit 0f535d4 into perry-mitchell:master May 9, 2021
@jstastny
Copy link
Contributor Author

Thanks for quickly merging both of my PRs and for the fast release of 4.4.0.

@jstastny jstastny deleted the collection-path branch May 10, 2021 10:18
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.

2 participants