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

Request a valid property for DAV opendir #9955

Merged
merged 1 commit into from
Jun 22, 2018
Merged

Conversation

rullzer
Copy link
Member

@rullzer rullzer commented Jun 21, 2018

Apperently Sabre and Onedrive (webdav endpoin) are not friends when requesting a single
404 property. I need to dig deeper on why this is. Anyways requesting a
valid property makes it work like a charm.

Signed-off-by: Roeland Jago Douma [email protected]

Apperently Sabre and Onedrive are not friends when requesting a single
404 property. I need to dig deeper on why this is. Anyways requesting a
valid property makes it work like a charm.

Signed-off-by: Roeland Jago Douma <[email protected]>
@codecov
Copy link

codecov bot commented Jun 21, 2018

Codecov Report

Merging #9955 into master will not change coverage.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #9955   +/-   ##
=========================================
  Coverage     52.08%   52.08%           
  Complexity    25944    25944           
=========================================
  Files          1645     1645           
  Lines         95828    95828           
  Branches       1290     1290           
=========================================
  Hits          49916    49916           
  Misses        45912    45912
Impacted Files Coverage Δ Complexity Δ
lib/private/Files/Storage/DAV.php 65.18% <100%> (ø) 159 <0> (ø) ⬇️

@rullzer rullzer merged commit 082646f into master Jun 22, 2018
@rullzer rullzer deleted the bug/noid/onedrive_propfind branch June 22, 2018 13:02
@MorrisJobke
Copy link
Member

What about the backport?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants