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

HTTP ReadTimeout/WriteTimeout causing unnecessary errors. #17219

Closed
reedwind opened this issue Jul 21, 2018 · 2 comments
Closed

HTTP ReadTimeout/WriteTimeout causing unnecessary errors. #17219

reedwind opened this issue Jul 21, 2018 · 2 comments

Comments

@reedwind
Copy link

Recently timeouts were added to the HTTP RPC server in #16859 and some arbitrary timeouts were chosen with a call for feedback.

For local hosts the timeouts are mostly fine, but if you run a node on a server ReadTimeout is much too low, as it's reliant on the client speed rather than the server. This is specifically an issue when trying to send large batched requests to get block or transaction information, which causes unnecessary timeout errors on slower connections.

I would recommend setting ReadTimeout to at least 15-30 seconds, and possibly also increasing WriteTimeout to 30-60 seconds to accommodate the large responses specifically from log requests which can often be in the megabytes. IdleTimeout is fine.

@stale
Copy link

stale bot commented Jul 22, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale
Copy link

stale bot commented Jul 22, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot closed this as completed Sep 2, 2019
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

1 participant