Skip to content

[ticket/17712] Verify thumbnail existence in attachment controller - #7036

Open
rubencm wants to merge 1 commit into
phpbb:masterfrom
rubencm:ticket/17712
Open

rubencm wants to merge 1 commit into
phpbb:masterfrom
rubencm:ticket/17712

Conversation

@rubencm

@rubencm rubencm commented Sep 17, 2026

Copy link
Copy Markdown
Member

Prepend thumbnail prefix before checking if the physical file exists in storage so missing thumbnails return 404 rather than triggering unhandled file exceptions.

PHPBB-17712

Checklist:

  • Correct branch: master for new features; 3.3.x for fixes
  • Tests pass
  • Code follows coding guidelines: master and 3.3.x
  • Commit follows commit message format

Tracker ticket:

https://tracker.phpbb.com/browse/PHPBB-12345

Prepend thumbnail prefix before checking if the physical file exists
in storage so missing thumbnails return 404 rather than triggering
unhandled file exceptions.

PHPBB-17712
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