[wip] id-based webdav requests #11180
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
JammingBen
changed the title
JammingBen
force-pushed
the
id-based-webdav-requests
branch
2 times, most recently
from
c1d2c3c to
b1abb65
Compare
|
JammingBen
force-pushed
the
id-based-webdav-requests
branch
7 times, most recently
from
32748a8 to
49f3265
Compare
Introduce id-based webdav requests for space and folder listings. This is the recommended way to work with the webdav API according to oCIS because it delivers better performance, especially with slower storages. Some of the webdav operations still need path handling though. Because of that, this change still "injects" full resources paths in the space loader. This can (hopefully) be removed once all operations work id-based or when we make the switch to Graph.
JammingBen
force-pushed
the
id-based-webdav-requests
branch
from
49f3265 to
414f588
Compare
JammingBen
changed the title
JammingBen
changed the title
|
Closing since we'll never merge this PR. It's only a reference for me what still needs to be done on the road to id-based webdav requests. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Introduce id-based webdav requests. This is the recommended way to work with the webdav API according to oCIS because it delivers better performance, especially with slower storages.
This is quite a big PR that tries to gather all relevant changes. The plan is to divide it into smaller parts and create separate PRs. I'm not 100% sure of we can (or should) fully switch to id-based webdav requests yet. But there are definitely parts of it that make sense to do now.
Related Issue
Types of changes