fix: Remove redundant pinning of futures in connection handler

freenet · Dec 26, 2024 · 61b7609 · 61b7609
1 parent d5f8d51
commit 61b7609
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/crates/core/src/transport/connection_handler.rs b/crates/core/src/transport/connection_handler.rs
@@ -291,8 +291,7 @@ impl<S: Socket> UdpPacketsListener<S> {
                             let task = tokio::spawn({
                                 let span = tracing::span!(tracing::Level::DEBUG, "gateway_connection");
                                 async move {
-                                    let pinned = Box::pin(gw_ongoing_connection);
-                                    match pinned.await {
+                                    match gw_ongoing_connection.await {
                                         Ok(result) => Ok(result),
                                         Err(error) => Err((error, remote_addr))
                                     }
@@ -381,8 +380,7 @@ impl<S: Socket> UdpPacketsListener<S> {
                     let task = tokio::spawn({
                         let span = span!(tracing::Level::DEBUG, "traverse_nat");
                         async move {
-                            let pinned = Box::pin(ongoing_connection);
-                            match pinned.await {
+                            match ongoing_connection.await {
                                 Ok(result) => Ok(result),
                                 Err(error) => Err((error, remote_addr))
                             }