Skip to content

[stable35] fix(files_sharing): don't abort share:list when orphaned shares exist - #64144

Open
backportbot[bot] wants to merge 1 commit into
stable35from
backport/62060/stable35
Open

[stable35] fix(files_sharing): don't abort share:list when orphaned shares exist#64144
backportbot[bot] wants to merge 1 commit into
stable35from
backport/62060/stable35

Conversation

@backportbot

@backportbot backportbot Bot commented Sep 9, 2026

Copy link
Copy Markdown

Backport of PR #62060

@backportbot
backportbot Bot requested a review from a team as a code owner September 9, 2026 12:54
@backportbot
backportbot Bot requested review from CarlSchwan, come-nc, icewind1991, joshtrichards, leftybournes and provokateurin and removed request for a team September 9, 2026 12:54
@backportbot backportbot Bot added this to the Nextcloud 35.0.1 milestone Sep 9, 2026
`occ share:list` resolved the source node of every share without
handling NotFoundException, so a single orphaned share (e.g. the source
file was deleted from disk externally, or is pending removal by
DeleteOrphanedSharesJob) aborted the whole command with "Node for share
not found" and produced no output at all.

Catch the exception in both places that resolve the share node: orphaned
shares are now listed with a null source-path, and the --parent filter
skips them instead of crashing.

Add unit tests for the command (previously untested), covering listing
with an orphaned share present and the --parent filter with and without
--recursive.

Closes #61949

Assisted-by: ClaudeCode:claude-fable-5
Signed-off-by: Fabio Fantoni <fabio.fantoni@m2r.biz>
@backportbot
backportbot Bot force-pushed the backport/62060/stable35 branch from b32b052 to 52dbfb7 Compare September 9, 2026 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants