Merge pull request maidsafe#2225 from joshuef/RemoveConnectionBadNode

feat(networking): remove NodeIssue::Connection
jacderida · Oct 11, 2024 · b772b89 · b772b89
2 parents 00b1fd3 + d2284d3
commit b772b89
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 9 deletions.
diff --git a/sn_networking/src/cmd.rs b/sn_networking/src/cmd.rs
@@ -44,8 +44,6 @@ const REPLICATION_TIMEOUT: Duration = Duration::from_secs(45);
 
 #[derive(Debug, Eq, PartialEq)]
 pub enum NodeIssue {
-    /// Connection issues observed
-    ConnectionIssue,
     /// Data Replication failed
     ReplicationFailure,
     /// Close nodes have reported this peer as bad

diff --git a/sn_networking/src/event/swarm.rs b/sn_networking/src/event/swarm.rs
@@ -7,8 +7,8 @@
 // permissions and limitations relating to use of the SAFE Network Software.
 
 use crate::{
-    cmd::LocalSwarmCmd, event::NodeEvent, multiaddr_is_global, multiaddr_strip_p2p,
-    relay_manager::is_a_relayed_peer, target_arch::Instant, NetworkEvent, Result, SwarmDriver,
+    event::NodeEvent, multiaddr_is_global, multiaddr_strip_p2p, relay_manager::is_a_relayed_peer,
+    target_arch::Instant, NetworkEvent, Result, SwarmDriver,
 };
 #[cfg(feature = "local")]
 use libp2p::mdns;
@@ -494,11 +494,6 @@ impl SwarmDriver {
                         .remove_peer(&failed_peer_id)
                     {
                         self.update_on_peer_removal(*dead_peer.node.key.preimage());
-
-                        self.handle_local_cmd(LocalSwarmCmd::RecordNodeIssue {
-                            peer_id: failed_peer_id,
-                            issue: crate::NodeIssue::ConnectionIssue,
-                        })?;
                     }
                 }
             }