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

Elapsed time timeout #562

Closed
shaicoleman opened this issue Aug 26, 2019 · 3 comments
Closed

Elapsed time timeout #562

shaicoleman opened this issue Aug 26, 2019 · 3 comments

Comments

@shaicoleman
Copy link

I believe at the moment it's only possibly to either specify a per operation timeout or a global timeout. Would be good if it were possible to combine both timeouts, e.g.
HTTP.timeout(global: 60, connect: 5, write: 2, read: 10).get 'http://example.com'

This is needed to handle servers that send unending responses or that send out data very slowly.

@ixti
Copy link
Member

ixti commented Aug 26, 2019

Hm... But global timeout is doing a bit different thing, it's basically timeout for connect + write + read. IN other words it won't solve your problem if I understand correctly.

@shaicoleman
Copy link
Author

In that case perhaps a different term is needed, e.g.

  • elapsed time timeout
  • overall timeout
  • total time timeout
  • cumulative time timeout

@shaicoleman shaicoleman changed the title Combine global timeout with per operation timeout Elapsed time timeout Aug 29, 2019
@tarcieri tarcieri pinned this issue Dec 13, 2023
@tarcieri
Copy link
Member

Closing in favor of #773

@tarcieri tarcieri closed this as not planned Won't fix, can't repro, duplicate, stale Dec 13, 2023
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

No branches or pull requests

3 participants