MOVE commands fail on some broken web servers when overwriting
Bug #1204727 reported by
Reagan Sanders
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Bazaar WebDAV plugin |
Fix Released
|
Undecided
|
Reagan Sanders |
Bug Description
MOVE commands that will result in an existing file being overwritten are issued without the Overwrite header being explicitly set. This complies with the RFC, as compliant servers must assume a value of T if the Overwrite header is absent.
Some paranoid non-complaint servers, however, will refuse to overwrite an existing file unless you explicitly set the Overwrite header to T, totally breaking pushes as-is. There isn't any real downside to sending the extra header, even when it isn't needed, and it will improve compatibility.
Related branches
lp://staging/~vexo/bzr-webdav/bzr-webdav
- Richard Wilbur: Approve
- Vincent Ladeuil: Needs Fixing
- Reagan Sanders (community): Needs Resubmitting
-
Diff: 117 lines (+35/-8)2 files modifiedtests/dav_server.py (+28/-2)
webdav.py (+7/-6)
Changed in bzr-webdav: | |
assignee: | nobody → Reagan Sanders (vexofp) |
status: | New → In Progress |
Changed in bzr-webdav: | |
status: | In Progress → Fix Released |
To post a comment you must log in.