Skip to content

ref(clients): Change defaults for timeouts#246

Merged
jan-auer merged 9 commits intomainfrom
ref/client-timeout-defaults
Dec 18, 2025
Merged

ref(clients): Change defaults for timeouts#246
jan-auer merged 9 commits intomainfrom
ref/client-timeout-defaults

Conversation

@jan-auer
Copy link
Member

Our read timeouts have been causing issues on the server side since they cause
the connection to drop, which can leave operations in a dangling state. This has
to be addressed separately, but also uncovered that a default read timeout is
likely not desired by use cases.

This changes timeout functionality in both clients equally:

  1. There is a default connect timeout of 100ms
  2. There is no default read timeout anymore
  3. The timeout / timeout_ms setting only affects read timeouts
  4. To set connection timeouts, one has to use the lower-level client config.

This is a BREAKING CHANGE

@jan-auer jan-auer requested a review from a team as a code owner December 17, 2025 16:28
@jan-auer jan-auer merged commit 46df2e8 into main Dec 18, 2025
24 of 25 checks passed
@jan-auer jan-auer deleted the ref/client-timeout-defaults branch December 18, 2025 13:23
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

Successfully merging this pull request may close these issues.

2 participants

Comments