vectordotdev / vector

A high-performance observability data pipeline.
https://vector.dev
Mozilla Public License 2.0
18.12k stars 1.6k forks source link

Upgrade the `http` crate to version 1.0.0 #19179

Open bruceg opened 12 months ago

bruceg commented 12 months ago

This requires updating several crates simultaneously, only some of which currently have updates to their http dependencies. I will update this issue as the dependency crates are bumped:

pront commented 11 months ago

Seems like https://crates.io/crates/http-types and https://crates.io/crates/hyper-proxy are on the latest version.

Also, https://crates.io/crates/tokio-tungstenite (PR) is another crate that seems to affected by the http upgrade.

bruceg commented 11 months ago

Yes, both http-types and hyper-proxy do not have a released version that upgrades their usage to http 1.0. So, we are using their latest versions, but upgrading the other crates without those breaks the build.

Agreed, at least part of the tokio-tungstenite breakage is caused by this issue. I'll add that to the list.

bruceg commented 10 months ago

Note that hyper-proxy appears to be unmaintained, as the last commit was 3 years ago, which may require converting Vector to use hyper-proxy2 (link) or handling proxying manually.

bruceg commented 10 months ago

Similarly, the last commit to http-types was 2 years ago, but replacing it will be more difficult as it is depended on by azure_core and wiremock.

bruceg commented 4 months ago

tonic now has released an updated version for the http/hyper 1.0 upgrade. https://github.com/hyperium/tonic/blob/master/CHANGELOG.md

That just leaves the two unmaintained crates, which will need to either be removed or forked and handled ourselves.