-
Notifications
You must be signed in to change notification settings - Fork 188
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
Default request timeouts #256
Comments
Breaking out the per-operation override capability out into #1732 since that requires a good bit of design work and refactoring. |
Defaults should be the following for now:
Taken from the current |
It looks like only the API timeouts have been implemented, so these defaults can't be set yet. Edit: From discussion with @Velfi, these timeouts don't need to get implemented. I will pare down the configuration interface to remove them for now, and will update this issue description with remaining work. |
TimeoutConfig
connect_timeout
toTimeoutConfig
and default it to 3100 millisecondsThe text was updated successfully, but these errors were encountered: