Skip to content

fix(dav): return NotFound when an ancestor's file info is unavailable - #66099

Open
lazerg wants to merge 1 commit into
nextcloud:masterfrom
lazerg:fix/issue-65248-directory-parent-false
Open

lazerg wants to merge 1 commit into
nextcloud:masterfrom
lazerg:fix/issue-65248-directory-parent-false

Conversation

@lazerg

@lazerg lazerg commented Oct 31, 2026

Copy link
Copy Markdown

Summary

When an ancestor is hidden by a Team folders ACL, View::getFileInfo() returns false and the Directory constructor in getNodeForPath() throws a TypeError, so WebDAV answers 500. It now throws NotFound like the other unreadable-ancestor case.

AI (if applicable)

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

Written with Claude Code (Claude Sonnet 5.5); I could not run the unit tests locally.

@lazerg
lazerg requested a review from a team as a code owner October 31, 2026 08:04
@lazerg
lazerg requested review from Altahrim, CarlSchwan, icewind1991 and salmart-dev and removed request for a team October 31, 2026 08:04
Assisted-by: ClaudeCode:claude-sonnet-5-5
Signed-off-by: lazerg <lazerg2@gmail.com>
@lazerg
lazerg force-pushed the fix/issue-65248-directory-parent-false branch from 1a2fa45 to 320d515 Compare October 31, 2026 08:04

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.

[Bug]: WebDAV TypeError (HTTP 500) in Directory::getNodeForPath when an ancestor is hidden by a Team folders ACL (Directory.php:571)

1 participant