-
Notifications
You must be signed in to change notification settings - Fork 8
Make the HTTP timeout configurable #13
Copy link
Copy link
Closed
Labels
area: httpNetwork calls, errors, timeouts and retriesNetwork calls, errors, timeouts and retriesenhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomers
Description
Activity
Metadata
Metadata
Assignees
Labels
area: httpNetwork calls, errors, timeouts and retriesNetwork calls, errors, timeouts and retriesenhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomers
Motivation
request_jsonuses a fixed 60 s timeout. Latency-sensitive agents may want 5 to 10 s and then fall back, while batch jobs may want more.Proposal
Add
Router(..., timeout=60), stored on the router and passed tojev.choose_via_*and on torequest_json(timeout=...).Done when
urlopen(assert with a mock)