search

eclipse-opendut / opendut

Test Electronic Control Units around the world in a transparent network.
https://opendut.eclipse.dev
Apache License 2.0
25 stars 15 forks source link

Upgrade networking stack dependencies #29

Open kKdH opened 10 months ago

kKdH commented 10 months ago

There is a new version of axum (our web application framework), we would like to update to. The update relates also to hyper, http, and reqwest which should be updated too. Due to several breaking changes the update is more complex.

mbfm commented 10 months ago

It looks like most conflicts happen, because the reqwest library isn't yet ready for using hyper and http 1.0: https://github.com/seanmonstar/reqwest/issues/2039

Trying to upgrade axum or hyper individually, forces those to use a new version of the http library, incompatible with what reqwest supports (0.2).

mbfm commented 10 months ago

The tonic library is not yet ready either: https://github.com/hyperium/tonic/issues/1579

mbfm commented 6 months ago

reqwest is now ready for upgrade, but tonic is still not ready.

A partial upgrade still fails due to two different versions of the http crate (0.2.12 and 1.1.0) being pulled in as dependencies.

mbfm commented 4 months ago

tonic is now also ready, as of version 0.12.0: https://github.com/hyperium/tonic/releases/tag/v0.12.0

mbfm commented 4 months ago

Upgrade attempted on branch issue-29-hyper-http-upgrade.

OpenTelemetry still seems to not be ready: https://github.com/open-telemetry/opentelemetry-rust/pull/1674

mbfm commented 4 months ago

OpenTelemetry is ready, now waiting for tracing-opentelemetry: https://github.com/tokio-rs/tracing-opentelemetry/issues/154

mbfm commented 3 months ago

tracing-opentelemetry is ready. Struggling with upgrading Axum for our gRPC+HTTP backend. The relevant example has not been updated yet, so might not yet be easily possible: https://github.com/tokio-rs/axum/blob/main/examples/rest-grpc-multiplex/src/main.rs

mbfm commented 2 days ago

reqwest-retry depends on the instant library which had an advisory issued, due to being unmaintained. We cannot upgrade reqwest-retry due to this issue.