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

Increase default OKHTTP Timeout #262

Closed
ProTip opened this issue Aug 9, 2019 · 0 comments
Closed

Increase default OKHTTP Timeout #262

ProTip opened this issue Aug 9, 2019 · 0 comments

Comments

@ProTip
Copy link
Contributor

ProTip commented Aug 9, 2019

The default timeout can cause issues when doing stuff like deleting executions. I think it defaults to 10 seconds and we have had people hitting read timeout errors deleting just 10 executions.

Raising this default to something dramatic like 10+ minutes with the current option to reduce as needed may be less surprising to users. Having a separate, lower connect timeout from the read timeout would probably be ideal.

gschueler added a commit that referenced this issue Aug 14, 2020
fix #262 increase default read/connect timeouts (10m/2m)
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