From 3ce2a13711db1c555a204ce89cc0aaa58d689562 Mon Sep 17 00:00:00 2001
From: h7x4 <h7x4@nani.wtf>
Date: Fri, 9 Jan 2026 17:36:04 +0900
Subject: [PATCH] server: demote a few debug log messages to trace

---
 src/server/session_handler.rs | 23 +++++++++++++----------
 src/server/supervisor.rs      | 13 ++++---------
 2 files changed, 17 insertions(+), 19 deletions(-)

diff --git a/src/server/session_handler.rs b/src/server/session_handler.rs
index d2a174e..b683fe6 100644
--- a/src/server/session_handler.rs
+++ b/src/server/session_handler.rs
@@ -76,7 +76,7 @@ pub async fn session_handler(
         }
     };
 
-    tracing::debug!("Validated peer UID: {}", uid);
+    tracing::trace!("Validated peer UID: {}", uid);
 
     let unix_user = match UnixUser::from_uid(uid) {
         Ok(user) => user,
@@ -137,7 +137,7 @@ pub async fn session_handler_with_unix_user(
 ) -> anyhow::Result<()> {
     let mut message_stream = create_server_to_client_message_stream(socket);
 
-    tracing::debug!("Requesting database connection from pool");
+    tracing::trace!("Requesting database connection from pool");
     let mut db_connection = match db_pool.read().await.acquire().await {
         Ok(connection) => connection,
         Err(err) => {
@@ -154,7 +154,7 @@ pub async fn session_handler_with_unix_user(
             return Err(err.into());
         }
     };
-    tracing::debug!("Successfully acquired database connection from pool");
+    tracing::trace!("Successfully acquired database connection from pool");
 
     let result = session_handler_with_db_connection(
         message_stream,
@@ -166,7 +166,7 @@ pub async fn session_handler_with_unix_user(
     )
     .await;
 
-    tracing::debug!("Releasing database connection back to pool");
+    tracing::trace!("Releasing database connection back to pool");
 
     result
 }
@@ -230,17 +230,20 @@ async fn handle_request(
     stream: &mut ServerToClientMessageStream,
 ) -> anyhow::Result<bool> {
     match &request {
-        Request::Exit => tracing::debug!("Received request: {:#?}", request),
+        Request::Exit => tracing::debug!("Request: exit"),
         Request::PasswdUser((db_user, _)) => tracing::debug!(
-            "Received request: {:#?}",
-            Request::PasswdUser((db_user.to_owned(), "<REDACTED>".to_string()))
+            "Request:\n{}",
+            serde_json::to_string_pretty(&Request::PasswdUser((
+                db_user.to_owned(),
+                "<REDACTED>".to_string()
+            )))?
         ),
         request => tracing::debug!("Request:\n{}", serde_json::to_string_pretty(request)?),
     }
 
     let affected_dbs = request.affected_databases();
     if !affected_dbs.is_empty() {
-        tracing::debug!(
+        tracing::trace!(
             "Affected databases: {}",
             affected_dbs.into_iter().map(|db| db.to_string()).join(", ")
         );
@@ -248,7 +251,7 @@ async fn handle_request(
 
     let affected_users = request.affected_users();
     if !affected_users.is_empty() {
-        tracing::debug!(
+        tracing::trace!(
             "Affected users: {}",
             affected_users.into_iter().map(|u| u.to_string()).join(", "),
         );
@@ -482,7 +485,7 @@ async fn handle_request(
 
     stream.send(response).await?;
     stream.flush().await?;
-    tracing::debug!("Successfully processed request");
+    tracing::trace!("Successfully processed request");
 
     Ok(true)
 }
diff --git a/src/server/supervisor.rs b/src/server/supervisor.rs
index e94d20a..71fc3ab 100644
--- a/src/server/supervisor.rs
+++ b/src/server/supervisor.rs
@@ -582,7 +582,6 @@ async fn listener_task(
             } => {
                 match accept_result {
                     Ok((conn, _addr)) => {
-
                         connection_counter.fetch_add(1, Ordering::Relaxed);
                         let conn_id = connection_counter.load(Ordering::Relaxed);
 
@@ -600,16 +599,12 @@ async fn listener_task(
                                 db_is_mariadb_clone,
                                 &*group_denylist_arc_clone.read().await,
                             ).await {
-                                Ok(()) => {}
-                                Err(e) => {
-                                    tracing::error!("Failed to run server: {}", e);
-                                }
+                                Ok(()) => {},
+                                Err(e) => tracing::error!("Session {} failed: {}", conn_id, e),
                             }
                         });
-                    }
-                    Err(e) => {
-                        tracing::error!("Failed to accept new connection: {}", e);
-                    }
+                    },
+                    Err(e) => tracing::error!("Failed to accept new connection: {}", e),
                 }
             }
         }