Skip to content

Give MKCOL the status vocabulary RFC 4918 defines - #984

Merged
slachiewicz merged 1 commit into
masterfrom
agent/webdav-mkcol-status-master
Sep 4, 2026
Merged

Give MKCOL the status vocabulary RFC 4918 defines#984
slachiewicz merged 1 commit into
masterfrom
agent/webdav-mkcol-status-master

Conversation

@slachiewicz

Copy link
Copy Markdown
Member

Master counterpart of #983, cherry-picked clean. See #630 for the report and #983 for the reasoning.

Verified on this branch: mvn install -pl wagon-providers/wagon-webdav-jackrabbit -am → 308 tests, 0 failures. spotless:check passes.

This change was created with AI assistance.

Only 409 means an ancestor is missing, so only 409 should send the walk
further back; treating every other answer that way turned a 403 or an
authentication failure into a confusing error about the top of the path.
On the way down, 405 means the collection is already there, which is as
good as having created it -- rejecting it failed uploads whenever a
collection appeared underneath us. Do not narrow either set back to
201-only; the walk is not atomic and the server is entitled to both.
@slachiewicz slachiewicz added the bug Something isn't working label Sep 4, 2026
@slachiewicz
slachiewicz marked this pull request as ready for review September 4, 2026 10:06
@slachiewicz
slachiewicz merged commit c0a7f25 into master Sep 4, 2026
12 checks passed
@slachiewicz
slachiewicz deleted the agent/webdav-mkcol-status-master branch September 4, 2026 11:33
@github-actions github-actions Bot added this to the 4.0.0-M1 milestone Sep 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant