Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding an RPC timeout parameter #23416

Closed
domohawk opened this issue Aug 19, 2021 · 3 comments
Closed

Adding an RPC timeout parameter #23416

domohawk opened this issue Aug 19, 2021 · 3 comments

Comments

@domohawk
Copy link

Currently the 5s RPC timeout is hardcoded. If we were to add a command line param to configure it, where should the config live?

As for why this might be useful, I'm bootstrapping initial contract state in another process using a call() via local RPC. Fetching a uint256[1000000] memory via a call() takes just under 5s, usually.

@holiman
Copy link
Contributor

holiman commented Aug 26, 2021

SGTM, this could live within the RPC config flags in the TOML config.

@jwasinger
Copy link
Contributor

Can this be closed? The main use-case for configurable timeout is for eth_call and tracing. Configurable timeout for eth_call was merged in #23645. Timeouts for tracing are mostly caused by sending the massive trace responses over http/websocket and the standardTraceBlockToFile is available.

@holiman
Copy link
Contributor

holiman commented Nov 4, 2021

Can this be closed?

Three weeks ago.
1,2, 3 closing!

@holiman holiman closed this as completed Nov 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants