From d116a434bf68f0c61656a1368ffe2ca6d916a544 Mon Sep 17 00:00:00 2001 From: Alexey Miroshkin Date: Tue, 27 Feb 2018 21:33:40 +0100 Subject: [PATCH] Log the reason of peer ban (#736) This PR fixed #455 by adding a few debug log messages --- p2p/src/peers.rs | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/p2p/src/peers.rs b/p2p/src/peers.rs index 72eceb761..d4642a286 100644 --- a/p2p/src/peers.rs +++ b/p2p/src/peers.rs @@ -442,9 +442,11 @@ impl ChainAdapter for Peers { self.adapter.transaction_received(tx) } fn block_received(&self, b: core::Block, peer_addr: SocketAddr) -> bool { + let hash = b.hash(); if !self.adapter.block_received(b, peer_addr) { // if the peer sent us a block that's intrinsically bad // they are either mistaken or manevolent, both of which require a ban + debug!(LOGGER, "Received a bad block {} from {}, the peer will be banned", hash, peer_addr); self.ban_peer(&peer_addr); false } else { @@ -452,9 +454,11 @@ impl ChainAdapter for Peers { } } fn compact_block_received(&self, cb: core::CompactBlock, peer_addr: SocketAddr) -> bool { + let hash = cb.hash(); if !self.adapter.compact_block_received(cb, peer_addr) { // if the peer sent us a block that's intrinsically bad // they are either mistaken or manevolent, both of which require a ban + debug!(LOGGER, "Received a bad compact block {} from {}, the peer will be banned", hash, &peer_addr); self.ban_peer(&peer_addr); false } else { @@ -493,6 +497,7 @@ impl ChainAdapter for Peers { ) -> bool { if !self.adapter.sumtrees_write(h, rewind_to_output, rewind_to_kernel, sumtree_data, peer_addr) { + debug!(LOGGER, "Received a bad sumtree data from {}, the peer will be banned", &peer_addr); self.ban_peer(&peer_addr); false } else {