ducks adds backports and deprecated features

blt · blt · commit 28b6db2966b8 · 2024-11-11T18:15:26.000-08:00
I'm attempting to upgrade hyper et al in our project. This is a delicate
business and I want to go slowly. I'm following the migration guide
one crate at a time.

Signed-off-by: Brian L. Troutwine &lt;brian.troutwine@datadoghq.com&gt;
diff --git a/integration/ducks/Cargo.toml b/integration/ducks/Cargo.toml
@@ -11,7 +11,7 @@ publish = false
 anyhow = "1.0"
 bytes = { workspace = true }
 entropy = "0.4"
-hyper = { version = "0.14", features = ["server"] }
+hyper = { version = "0.14", features = ["server", "backports", "deprecated"] }
 once_cell = { workspace = true }
 shared = { path = "../shared" }
 sketches-ddsketch = "0.3"
diff --git a/integration/ducks/src/main.rs b/integration/ducks/src/main.rs
@@ -16,7 +16,7 @@
 use anyhow::Context;
 use bytes::BytesMut;
 use hyper::{
-    body,
+    body::HttpBody,
     server::conn::{AddrIncoming, AddrStream},
     service::{make_service_fn, service_fn},
     Body, Method, Request, StatusCode,
@@ -127,7 +127,7 @@ impl From<&SocketCounters> for SocketMetrics {
 #[tracing::instrument(level = "trace")]
 async fn http_req_handler(req: Request<Body>) -> Result<hyper::Response<Body>, hyper::Error> {
     let (parts, body) = req.into_parts();
-    let body = body::to_bytes(body).await?;
+    let body = body.collect().await?.to_bytes();
 
     {
         let metric = HTTP_COUNTERS.get().expect("HTTP_COUNTERS not initialized");
