From f3a25a79fc3a47f3becedfc23d0f85dd3356fc8a Mon Sep 17 00:00:00 2001
From: Lucas Balieiro <github@lucasbalieiro.dev>
Date: Thu, 14 Aug 2025 12:22:11 -0400
Subject: [PATCH] handle double client removal by logging instead of panicking

---
 src/sv2_handlers/mining_server_handler.rs | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/src/sv2_handlers/mining_server_handler.rs b/src/sv2_handlers/mining_server_handler.rs
index 71da2a6..6ba0ec5 100644
--- a/src/sv2_handlers/mining_server_handler.rs
+++ b/src/sv2_handlers/mining_server_handler.rs
@@ -307,7 +307,16 @@ impl Sv2MiningServerHandler for PlebLotteryMiningServerHandler {
         let hashrate = {
             let mut hash = 0.0;
             let clients_guard = self.clients.read().await;
-            let client = clients_guard.get(&client_id).unwrap();
+            let client = match clients_guard.get(&client_id) {
+                Some(c) => c,
+                None => {
+                    info!(
+                        "Client {} not found in clients list — assuming already dropped.",
+                        client_id
+                    );
+                    return;
+                }
+            };
             let client_guard = client.read().await;
 
             // Calculate hashrate from standard channels