Skip to content

rpc: add --rpc.http-body-limit flag for HTTP request body size - #35224

Merged
MariusVanDerWijden merged 1 commit into
ethereum:masterfrom
skylenet:http-body-limit-flag
Jul 6, 2026
Merged

rpc: add --rpc.http-body-limit flag for HTTP request body size#35224
MariusVanDerWijden merged 1 commit into
ethereum:masterfrom
skylenet:http-body-limit-flag

Conversation

@skylenet

@skylenet skylenet commented Jun 26, 2026

Copy link
Copy Markdown
Member

The HTTP JSON-RPC server caps request bodies at a hardcoded 5 MB (defaultBodyLimit), with no way to raise it. EEST bloatnet depth benchmarks post worst-case batch requests larger than that during fill-stateful, which the server rejects with 413 Request Entity Too Large.

This adds node.Config.HTTPBodyLimit (default 5 MB), wired through to the HTTP/WS server config, and exposes it via --rpc.http-body-limit (value in megabytes, default 5). Default behaviour is unchanged.

@skylenet
skylenet requested a review from fjl as a code owner June 26, 2026 08:36
@MariusVanDerWijden

Copy link
Copy Markdown
Member

I think I remember that when we introduced it we hardcoded it deliberately since there wasn't really any use case for bigger http requests. Since that changed, we should just do this

@skylenet
skylenet changed the base branch from bal-devnet-7 to master June 26, 2026 10:15
The HTTP JSON-RPC server caps request bodies at a hardcoded 5 MB
(defaultBodyLimit), with no way to raise it. EEST bloatnet depth
benchmarks post worst-case batch requests larger than that during
fill-stateful, which the server rejects with 413 Request Entity Too
Large. Add a node.Config.HTTPBodyLimit (default 5 MB), wired to the
HTTP/WS server config, and expose it via --rpc.http-body-limit (value in
megabytes, default 5). Default behaviour is unchanged.
@skylenet
skylenet force-pushed the http-body-limit-flag branch from 9915376 to 2a5d898 Compare June 26, 2026 10:16
@MariusVanDerWijden MariusVanDerWijden added this to the 1.17.5 milestone Jun 29, 2026
@fjl fjl removed the status:triage label Jun 30, 2026
@MariusVanDerWijden
MariusVanDerWijden merged commit 7c1959b into ethereum:master Jul 6, 2026
20 of 23 checks passed
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.

3 participants