improve logging to understand why connection fetching was panicing

sigp · Jan 8, 2024 · 5f43d26 · 5f43d26
1 parent b64bb06
commit 5f43d26
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/protocols/gossipsub/src/behaviour.rs b/protocols/gossipsub/src/behaviour.rs
@@ -3037,7 +3037,7 @@ where
         // Add the new connection
         connected_peer.connections.push(connection_id);
 
-        tracing::debug!(%peer_id, "Established inbound connection to peer");
+        tracing::info!(%peer_id, %connection_id, "Established inbound connection to peer");
         Ok(Handler::new(
             self.config.protocol_config(),
             connected_peer.sender.new_receiver(),
@@ -3068,7 +3068,7 @@ where
         // Add the new connection
         connected_peer.connections.push(connection_id);
 
-        tracing::debug!(%peer_id, "Established outbound connection to peer");
+        tracing::info!(%peer_id, %connection_id, "Established inbound connection to peer");
         Ok(Handler::new(
             self.config.protocol_config(),
             connected_peer.sender.new_receiver(),
@@ -3284,11 +3284,14 @@ where
                         if let Some(index) =
                             peer.connections.iter().position(|v| v == &connection_id)
                         {
+                            tracing::info!(%peer_id, %connection_id, "Removed connection from connected peers");
                             peer.connections.remove(index);
                         }
 
                         if !peer.connections.is_empty() {
                             self.connected_peers.insert(peer_id, peer);
+                        } else {
+                            tracing::info!(%peer_id, %connection_id, "Removed peer from connected peers");
                         }
                     }
                 }