Skip to content

fix: Catch expected NotFoundException#61917

Open
kesselb wants to merge 1 commit into
masterfrom
bug/noid/catch-exception
Open

fix: Catch expected NotFoundException#61917
kesselb wants to merge 1 commit into
masterfrom
bug/noid/catch-exception

Conversation

@kesselb

@kesselb kesselb commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Summary

The method was catching the wrong exception.

  • Create public share link for a folder
  • Copy the share token
  • Send http request like below (set dir to a value that wont exist)
  • 💥
POST https://server99.internal/index.php/apps/files_sharing/shareinfo?dir=test123
Content-Type: application/x-www-form-urlencoded

t=share_token

Checklist

AI (if applicable)

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

Assisted-by: ClaudeCode:claude-sonnet-5
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
@kesselb kesselb requested a review from a team as a code owner July 8, 2026 15:51
@kesselb kesselb requested review from CarlSchwan, icewind1991, leftybournes and salmart-dev and removed request for a team July 8, 2026 15:51
@kesselb kesselb self-assigned this Jul 8, 2026
@kesselb kesselb added bug 3. to review Waiting for reviews and removed AI assisted labels Jul 8, 2026
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