-
Notifications
You must be signed in to change notification settings - Fork 12
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
Did we introduce protocol.[webdav|options].URI accidentally #127
Comments
@glpatcern what do you think? |
Indeed the idea was at the time to not introduce a breaking change, and - without yet introducing a standard way to "announce" and consume announcements (the discovery endpoint was not yet part of the standard...) - the way to disentangle was the
In the current context, also in view of #121, that makes sense. And eventually we can deprecate the |
Reviewing this as part of #131, in fact we have Therefore, the capability the Receiving Server should announce is to be able to process a |
We can announce this inside the |
I was thinking about the URI property in WebDAV protocol details, and it occurred to me that
I traced it back to c54058b#diff-9cfca4a1b73e1e28e30fb9b0b984aad6d4caaf0819c61ed40ad338600531f745R327 which was part of the addition of multi-protocol as an optional feature, in the PR that fixed #56.
So I think:
URI
for WebDAV SHOULD announce thisURI
unless the Receiving Server has announced it understands itThe text was updated successfully, but these errors were encountered: