Skip to content

[stable34] fix(share): Avoid crash when share owner is not found - #62798

Open
backportbot[bot] wants to merge 1 commit into
stable34from
backport/62791/stable34
Open

[stable34] fix(share): Avoid crash when share owner is not found#62798
backportbot[bot] wants to merge 1 commit into
stable34from
backport/62791/stable34

Conversation

@backportbot

@backportbot backportbot Bot commented Aug 3, 2026

Copy link
Copy Markdown

Backport of PR #62791

@backportbot
backportbot Bot requested a review from a team as a code owner August 3, 2026 16:13
@backportbot
backportbot Bot requested review from Altahrim, come-nc, icewind1991, nfebe and sorbaugh and removed request for a team August 3, 2026 16:13
@backportbot backportbot Bot added bug 3. to review Waiting for reviews labels Aug 3, 2026
@backportbot backportbot Bot added this to the Nextcloud 34.0.3 milestone Aug 3, 2026
IShare::getNode is documented to throw only NotFoundException, so catch
 UserNotFoundException from the level below and wrap it. This avoids
 crashes from share api controller when listing shares and one of them
 is broken because its owner has vanished from the backend.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc
come-nc force-pushed the backport/62791/stable34 branch from 28beadc to b05a456 Compare August 3, 2026 16:21

@come-nc come-nc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adapted to 34. Should backport this version to older releases.

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.

1 participant