fix: clear up failed connection attempts (#425)

Not clearing them up restricts from repeated connection attempts.

Author: xdustinface

dash-spv/src/network/manager.rs

@@ -301,6 +301,7 @@ impl PeerNetworkManager {
                         }
                         Err(e) => {
                             log::warn!("Handshake failed with {}: {}", addr, e);
+                            pool.remove_peer(&addr).await;
                             // Update reputation for handshake failure
                             reputation_manager
                                 .update_reputation(
@@ -316,6 +317,7 @@ impl PeerNetworkManager {
                 }
                 Err(e) => {
                     log::debug!("Failed to connect to {}: {}", addr, e);
+                    pool.remove_peer(&addr).await;
                     // Minor reputation penalty for connection failure
                     reputation_manager
                         .update_reputation(

dash-spv/src/network/pool.rs

@@ -61,8 +61,9 @@ impl PeerPool {
         Ok(())
     }
 
-    /// Remove a peer from the pool
+    /// Remove a peer from the pool and clear connecting state
     pub async fn remove_peer(&self, addr: &SocketAddr) -> Option<Arc<RwLock<Peer>>> {
+        self.connecting.write().await.remove(addr);
         let removed = self.peers.write().await.remove(addr);
         if removed.is_some() {
             log::info!("Removed peer {}", addr);