Using tonic for clients still pulls in extra dependencies

[djc]

From my comment in #959 (comment):

I finally tried using #960 and I don't think it's an adequate fix. First of all, the tls feature still pulls in transport, so by default if you pull in tls or one of the features that depend on it, you'll still get the server-side dependencies. That part can actually be worked around by depending on the other required features directly, but then I noticed that the entire tonic::transport module is unavailable if the transport feature is disabled, so I don't have access to things like Error, Channel or Endpoint from that module.

Should I submit a PR replacing the use of transport internally with channel for client-side or both-side items?

[LucioFranco]

Ugh yeah seems like a mess, honestly, I would just use tonic without the transport module. Its still a bit of a mess but will allow you better fine grained control over what you pull in. Is this change for something internal or is it in a public repo?

[djc]

Something internal, though it's probably also relevant for parts of opentelemetry-rust (notably, opentelemetry-stackdriver).

[LucioFranco]

Do you think migrating to using hyper is feasible?

[LucioFranco]

I wrote this PR that should make using hyper directly much easier #1017

[djc]

Is there any example code for using tonic without the transport stuff?

[LucioFranco]

@djc the rustls examples under the tls folder has examples of using hyper. In general, it should just work, except servers don't provide the make service portion.