Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade networking stack dependencies #29

Closed
kKdH opened this issue Jan 15, 2024 · 9 comments
Closed

Upgrade networking stack dependencies #29

kKdH opened this issue Jan 15, 2024 · 9 comments
Assignees
Labels
component: carl Mostly related to CARL. type: enhancement Fulfills a need or requirement by improving an existing functionality.

Comments

@kKdH
Copy link
Member

kKdH commented Jan 15, 2024

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.

@kKdH kKdH added component: carl Mostly related to CARL. type: enhancement Fulfills a need or requirement by improving an existing functionality. labels Jan 15, 2024
@github-project-automation github-project-automation bot moved this to Backlog in openDuT Jan 15, 2024
@mbfm
Copy link
Contributor

mbfm commented Jan 16, 2024

It looks like most conflicts happen, because the reqwest library isn't yet ready for using hyper and http 1.0: seanmonstar/reqwest#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
Copy link
Contributor

mbfm commented Jan 17, 2024

The tonic library is not yet ready either: hyperium/tonic#1579

@mbfm
Copy link
Contributor

mbfm commented Apr 22, 2024

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
Copy link
Contributor

mbfm commented Jul 12, 2024

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

@mbfm
Copy link
Contributor

mbfm commented Jul 12, 2024

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

OpenTelemetry still seems to not be ready: open-telemetry/opentelemetry-rust#1674

@mbfm mbfm moved this from Backlog to Ready in openDuT Jul 15, 2024
@mbfm
Copy link
Contributor

mbfm commented Jul 17, 2024

OpenTelemetry is ready, now waiting for tracing-opentelemetry: tokio-rs/tracing-opentelemetry#154

@mbfm
Copy link
Contributor

mbfm commented Aug 6, 2024

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 mbfm changed the title Axum 0.7 update Upgrade networking stack Nov 13, 2024
@mbfm
Copy link
Contributor

mbfm commented Nov 13, 2024

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.

@mbfm mbfm changed the title Upgrade networking stack Upgrade networking stack dependencies Nov 14, 2024
@mbfm mbfm self-assigned this Nov 20, 2024
@mbfm mbfm moved this from In progress to In review in openDuT Dec 2, 2024
@mbfm
Copy link
Contributor

mbfm commented Dec 2, 2024

Implemented in 4e8c2e1 and following commits.

@mbfm mbfm closed this as completed Dec 2, 2024
@mbfm mbfm moved this from In review to Done in openDuT Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: carl Mostly related to CARL. type: enhancement Fulfills a need or requirement by improving an existing functionality.
Projects
Archived in project
Development

No branches or pull requests

2 participants