Skip to content

fix: handle oversized inputs in imports, CSV and printer previews - #606

Merged
u8array merged 3 commits into
u8array:mainfrom
skovkode:fix/cap-untrusted-sizes
Nov 6, 2026
Merged

u8array merged 3 commits into
u8array:mainfrom
skovkode:fix/cap-untrusted-sizes

Conversation

@skovkode

@skovkode skovkode commented Nov 5, 2026

Copy link
Copy Markdown
Collaborator

Fix crashes from oversized ^GF imports, long headerless CSV files and compressed printer replies.

Cap decompressed printer replies at 2 MB. Labels over roughly 11 inches at 4 inches wide and 600 dpi now show "reply could not be read".

Reject uncompressed graphics over 2 MB and clear previous uploads at the same path on rejection.

@skovkode
skovkode requested a review from u8array November 5, 2026 21:00

@u8array u8array left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thanks, this looks good! I noticed a few other issues during the review that are already present on main. I'll address those in a follow-up.

@u8array
u8array merged commit b8cc656 into u8array:main Nov 6, 2026
2 checks passed
@skovkode
skovkode deleted the fix/cap-untrusted-sizes branch November 6, 2026 06:14
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.

2 participants