From cb4b8a78dc8c3fc05c9a4433babcdebf1df62b61 Mon Sep 17 00:00:00 2001
From: h7x4 <h7x4@nani.wtf>
Date: Sat, 3 Jan 2026 23:04:04 +0900
Subject: [PATCH] session_handler: don't clone request before tracing

---
 src/server/session_handler.rs | 17 ++++++-----------
 1 file changed, 6 insertions(+), 11 deletions(-)

diff --git a/src/server/session_handler.rs b/src/server/session_handler.rs
index c1dc6d1..892ec29 100644
--- a/src/server/session_handler.rs
+++ b/src/server/session_handler.rs
@@ -173,18 +173,13 @@ async fn session_handler_with_db_connection(
             }
         };
 
-        // TODO: don't clone the request
-        let request_to_display = match &request {
-            Request::PasswdUser((db_user, _)) => {
+        match &request {
+            Request::Exit => tracing::debug!("Received request: {:#?}", request),
+            Request::PasswdUser((db_user, _)) => tracing::info!(
+                "Received request: {:#?}",
                 Request::PasswdUser((db_user.to_owned(), "<REDACTED>".to_string()))
-            }
-            request => request.to_owned(),
-        };
-
-        if request_to_display == Request::Exit {
-            tracing::debug!("Received request: {:#?}", request_to_display);
-        } else {
-            tracing::info!("Received request: {:#?}", request_to_display);
+            ),
+            request => tracing::info!("Received request: {:#?}", request),
         }
 
         let response = match request {