Merge pull request #4813 from eval-exec/exec/fix-log

ckb-network: print remove public_addr log when it really removed
nervosnetwork · Feb 20, 2025 · 0f34d1e · 0f34d1e
2 parents b05727c + ef0f427
commit 0f34d1e
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/network/src/network.rs b/network/src/network.rs
@@ -635,11 +635,12 @@ impl ServiceHandle for EventHandler {
                         debug!("DialerError({}) {}", address, error);
                     }
                 }
-                info!(
-                    "Dial {} failed, remove it from network_state.public_addrs",
-                    address
-                );
-                public_addrs.remove(&address);
+                if public_addrs.remove(&address) {
+                    info!(
+                        "Dial {} failed, remove it from network_state.public_addrs",
+                        address
+                    );
+                }
                 self.network_state.dial_failed(&address);
             }
             ServiceError::ProtocolError {