Skip to content

🐛 fix(sandbox): fix large attachment transfering between host and container - #34

Merged
nick-cjyx9 merged 1 commit into
mainfrom
fix/large-attachment
Aug 21, 2026
Merged

nick-cjyx9 merged 1 commit into
mainfrom
fix/large-attachment

Conversation

@fressive

Copy link
Copy Markdown
Collaborator

Chunk file transfers above 32 MiB so base64 payloads stay far below the V8 string limit (0x1fffffe8 chars); attachments over ~384 MiB previously crashed Solver creation with "Cannot create a string longer than 0x1fffffe8 characters". Large replaceFile stays atomic via staged temp file + rename. Attachment unchanged-check now compares size + SHA-256 instead of reading the whole file back (readBinaryFile is capped by the 8 MiB exec output limit, so files >= 6 MiB were always rewritten).

…tainer

Chunk file transfers above 32 MiB so base64 payloads stay far below the
V8 string limit (0x1fffffe8 chars); attachments over ~384 MiB previously
crashed Solver creation with \"Cannot create a string longer than
0x1fffffe8 characters\". Large replaceFile stays atomic via staged temp
file + rename. Attachment unchanged-check now compares size + SHA-256
instead of reading the whole file back (readBinaryFile is capped by the
8 MiB exec output limit, so files >= 6 MiB were always rewritten).
@nick-cjyx9
nick-cjyx9 merged commit 4934212 into main Aug 21, 2026
2 checks passed
@fressive
fressive deleted the fix/large-attachment branch August 21, 2026 18:27
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