From e3854173d720fbbc4f98a7939ac667fcac007ec1 Mon Sep 17 00:00:00 2001
From: Mroik <mroik@delayed.space>
Date: Sun, 12 Apr 2026 03:23:32 +0200
Subject: Move logging for premature TCP termination

It logs anytime we break out of the loop, therefore logging a connection
closed by remote regardless of what happened.

Move logging to closed connection check and make it a warn instead of an
info.

Signed-off-by: Mroik <mroik@delayed.space>
---
 src/smtp_server.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/smtp_server.rs b/src/smtp_server.rs
index 42e5cf4..9e313f4 100644
--- a/src/smtp_server.rs
+++ b/src/smtp_server.rs
@@ -93,6 +93,7 @@ impl SessionHandler {
             //
             // TODO: Handle non UTF-8 encoding.
             if reader.read_line(&mut buffer).await? == 0 {
+                log::warn!("Connection closed by {}", self.addr);
                 break;
             }
             log::debug!("Received '{}' from '{}'", buffer.trim(), self.addr);
@@ -125,7 +126,6 @@ impl SessionHandler {
             }
         }
 
-        log::info!("Connection closed by {}", self.addr);
         Ok(())
     }
 
-- 
cgit v1.3