search

TrueLayer / ginepro

A client-side gRPC channel implementation for tonic
Apache License 2.0
127 stars 24 forks source link

Add connection timeout #38

Closed ThomWright closed 1 year ago

ThomWright commented 1 year ago

Adds a connect_timeout() option to LoadBalancedChannelBuilder.

Relevant to #37.

Note: hyperium/tonic#1215 will need merging first for this to work.

ThomWright commented 1 year ago

Do we want some safe defaults here? If I specify just a timeout but no connect_timeout, should the connect_timeout be assumed to be the same as timeout?

I like this idea. Done.

ThomWright commented 1 year ago

I apparently don't have write access to merge 🥺