tonic icon indicating copy to clipboard operation
tonic copied to clipboard

Remove timeout handling from `Endpoint::connect_with_connector`

Open tottoto opened this issue 2 years ago • 0 comments

Feature Request

Crates

  • tonic

Motivation

Reduces transport's responsibility.

Proposal

[!WARNING] This proposal includes a breaking change.

Removes timeout handling from tonic::transport::channel::Endpoint::connect_with_connector method.

By this proposal (and #1497), tonic can remove hyper-timeout from dependencies. This simplifies tonic's dependency tree, responsibility and implementations. And also it makes easier a little the future migrating to hyper 1.0.

tonic user can apply timeout as of the timeout example.

Alternatives

tottoto avatar Oct 22 '23 06:10 tottoto