networking: Refactor the code.

- add comments
autonomys · Oct 10, 2023 · 82ee98a · 82ee98a
1 parent db8cabb
commit 82ee98a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/crates/subspace-networking/src/constructor.rs b/crates/subspace-networking/src/constructor.rs
@@ -235,7 +235,9 @@ pub struct Config<LocalRecordProvider> {
     pub special_target_connections: u32,
     /// Addresses to bootstrap Kademlia network
     pub bootstrap_addresses: Vec<Multiaddr>,
-    /// Kademlia mode.
+    /// Kademlia mode. The default value is set to Static(Client). The peer won't add its address
+    /// to other peers` Kademlia routing table. Changing this behaviour implies that a peer can
+    /// provide pieces to others.
     pub kademlia_mode: KademliaMode,
     /// Known external addresses to the local peer. The addresses will be added on the swarm start
     /// and enable peer to notify others about its reachable address.

diff --git a/crates/subspace-networking/src/node_runner.rs b/crates/subspace-networking/src/node_runner.rs
@@ -1195,7 +1195,7 @@ where
         if let AutonatEvent::StatusChanged { old, new } = event {
             info!(?old, ?new, "Public address status changed.");
 
-            if let KademliaMode::Dynamic { .. } = self.kademlia_mode {
+            if matches!(self.kademlia_mode, KademliaMode::Dynamic { .. }) {
                 let mode = match &new {
                     NatStatus::Public(address) => {
                         if is_global_address_or_dns(address)