Bound decoded screenshot pixels before mobile evidence preview #838

Merged
timmy merged 1 commits from timmy/837-bound-decoded-screenshot-pixels into main 2026-08-14 16:37:56 +00:00
Owner

Closes #837

What changed

  • inspect every mobile evidence image before preview
  • cap decoded images at 12 MiPixels and 8192px per dimension
  • combine pixel, dimension, and 2 MB encoded-size limits during resize
  • preserve filename/type and close decoded bitmap resources

Tests

  • RED confirmed for compressed 24 MP input and pre-preview inspection
  • python3 -m pytest tests/test_issue_attachment_ui.py -q (32 passed)
  • python3 -m pytest tests/ -q (1577 passed)
Closes #837 ## What changed - inspect every mobile evidence image before preview - cap decoded images at 12 MiPixels and 8192px per dimension - combine pixel, dimension, and 2 MB encoded-size limits during resize - preserve filename/type and close decoded bitmap resources ## Tests - RED confirmed for compressed 24 MP input and pre-preview inspection - `python3 -m pytest tests/test_issue_attachment_ui.py -q` (32 passed) - `python3 -m pytest tests/ -q` (1577 passed)
timmy added 1 commit 2026-08-14 16:35:38 +00:00
feat: bound decoded screenshot pixels (Closes #837)
All checks were successful
CI / lint (pull_request) Successful in 1m46s
CI / build-release (pull_request) Successful in 6s
CI / release-candidate (pull_request) Has been skipped
a476170fca
timmy merged commit 04f06042b8 into main 2026-08-14 16:37:56 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: stackchain/stackchain-dashboard#838
No description provided.