Skip to content

fix(dav): keep full paths when refreshing nodes - #66384

Open
mustafa519 wants to merge 1 commit into
nextcloud:masterfrom
mustafa519:codex/dav-refresh-absolute-path
Open

mustafa519 wants to merge 1 commit into
nextcloud:masterfrom
mustafa519:codex/dav-refresh-absolute-path

Conversation

@mustafa519

@mustafa519 mustafa519 commented Nov 6, 2026 •

Copy link
Copy Markdown
  • Resolves: N/A

Summary

When WebDAV corrects a cached file size during a download, it refreshes the node. The refresh drops the user part of the path, so the encryption check can fail to find the parent folder and return HTTP 500.

Use the same path conversion as the constructor when refreshing files and folders. This follows up on the review of #36774.

Both new cases reproduce the missing path prefix on master. With the fix, all 571 DAV connector tests pass.

TODO

  • Add the DCO sign-off to the commit.

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Codex helped prepare the patch and regression tests.

Assisted-by: Codex:gpt-6.1-sol
@mustafa519
mustafa519 requested a review from a team as a code owner November 6, 2026 04:51
@mustafa519
mustafa519 requested review from Altahrim and leftybournes and removed request for a team November 6, 2026 04:51

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant