Skip to content

fix(sandbox-proxy): raise proxied body/response cap to 20MB - #5353

Merged
pedrofrxncx merged 1 commit into
mainfrom
fix/sandbox-proxy-bump-body-cap-20mb
Jul 28, 2026
Merged

fix(sandbox-proxy): raise proxied body/response cap to 20MB#5353
pedrofrxncx merged 1 commit into
mainfrom
fix/sandbox-proxy-bump-body-cap-20mb

Conversation

@pedrofrxncx

@pedrofrxncx pedrofrxncx commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator

Follow-up to #5330 — bumps FORWARDED_BODY_MAX_BYTES from 10MB to 20MB.

That constant bounds both sides of the sandbox-proxy routes: the forwardedBodyLimit on incoming client bodies and the readBoundedText cap on daemon/preview responses. 10MB was tripping on legitimate payloads (large git diffs, chatty exec stdout, big preview pages); 20MB still bounds memory per in-flight request.

No other change — the cap is a single constant, tests are unaffected.

Verified: bun test apps/api/src/api/routes/sandbox-proxy.test.ts (10 pass), bun run fmt.


Summary by cubic

Raised the sandbox-proxy cap from 10MB to 20MB for both incoming request bodies and daemon/preview responses to stop valid large payloads from being rejected while keeping per-request memory bounded.

Written for commit 5a6f219. Summary will update on new commits.

Review in cubic

@pedrofrxncx
pedrofrxncx enabled auto-merge (squash) July 28, 2026 17:33
@pedrofrxncx
pedrofrxncx merged commit c7d80df into main Jul 28, 2026
16 checks passed
@pedrofrxncx
pedrofrxncx deleted the fix/sandbox-proxy-bump-body-cap-20mb branch July 28, 2026 17:38
decocms Bot pushed a commit that referenced this pull request Jul 28, 2026
PR: #5353 fix(sandbox-proxy): raise proxied body/response cap to 20MB
Bump type: patch

- decocms (apps/api/package.json): 4.139.17 -> 4.139.18

Deploy-Scope: server
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