From 27aec1962c7cc82f20648d257b95f94906909724 Mon Sep 17 00:00:00 2001 From: Paul Hauner Date: Tue, 13 Jul 2021 01:02:03 +0000 Subject: [PATCH] Add more detail to "Prior attestation known" log (#2447) ## Issue Addressed NA ## Proposed Changes Adds more detail to the log when an attestation is ignored due to a prior one being known. This will help identify which validators are causing the issue. ## Additional Info NA --- .../network/src/beacon_processor/worker/gossip_methods.rs | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/beacon_node/network/src/beacon_processor/worker/gossip_methods.rs b/beacon_node/network/src/beacon_processor/worker/gossip_methods.rs index 96a4d86c8..fcf63fc27 100644 --- a/beacon_node/network/src/beacon_processor/worker/gossip_methods.rs +++ b/beacon_node/network/src/beacon_processor/worker/gossip_methods.rs @@ -752,7 +752,10 @@ impl Worker { return; } - AttnError::PriorAttestationKnown { .. } => { + AttnError::PriorAttestationKnown { + validator_index, + epoch, + } => { /* * We have already seen an attestation from this validator for this epoch. * @@ -763,6 +766,8 @@ impl Worker { "Prior attestation known"; "peer_id" => %peer_id, "block" => %beacon_block_root, + "epoch" => %epoch, + "validator_index" => validator_index, "type" => ?attestation_type, ); // We still penalize the peer slightly. We don't want this to be a recurring