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

CLI timeout is too low #2637

Closed
matti opened this issue Jul 28, 2017 · 2 comments · Fixed by #2671
Closed

CLI timeout is too low #2637

matti opened this issue Jul 28, 2017 · 2 comments · Fixed by #2671
Assignees
Milestone

Comments

@matti
Copy link
Contributor

matti commented Jul 28, 2017

Could be a few seconds longer and it wouldn't hurt (eg. make the client to be stuck for 20-30s like it used to be with a longer timeout)

 [error] Excon::Errors::Timeout : connect_write timeout reached
@jakolehm jakolehm added this to the 1.4.0 milestone Jul 28, 2017
@jakolehm
Copy link
Contributor

10s might be better default than current 5.

@SpComb
Copy link
Contributor

SpComb commented Jul 28, 2017

The default EXCON_CONNECT_TIMEOUT is 5 seconds, which is a bit optimistic. You can override that as an env, but the default could be something like 10s.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants