feat: Use http::Extensions directly

hyperium / tonic

A native gRPC client & server implementation with async/await support.

https://docs.rs/tonic

MIT License

9.35k stars 957 forks source link

feat: Use http::Extensions directly #1710

Closed tottoto closed 3 weeks ago

tottoto commented 1 month ago

Motivation

Makes tonic's implementations thinner.

As tonic's own Extensions type do not anything than the wrapper type of http::Extensions, there seems to be no advantage other than not being effected from changes by http::Extensions. This benefit might be small because the http crate's stability term is longer than tonic's one.

Solution

Uses http::Extensions directly.

djc commented 1 month ago

This makes sense to me but I'd like to review/merge it after #1670 is merged.