Skip to content

Commit

Permalink
Add more detail to "Prior attestation known" log (#2447)
Browse files Browse the repository at this point in the history
## 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
  • Loading branch information
paulhauner committed Jul 13, 2021
1 parent 20fce11 commit 27aec19
Showing 1 changed file with 6 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -752,7 +752,10 @@ impl<T: BeaconChainTypes> Worker<T> {

return;
}
AttnError::PriorAttestationKnown { .. } => {
AttnError::PriorAttestationKnown {
validator_index,
epoch,
} => {
/*
* We have already seen an attestation from this validator for this epoch.
*
Expand All @@ -763,6 +766,8 @@ impl<T: BeaconChainTypes> Worker<T> {
"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
Expand Down

0 comments on commit 27aec19

Please sign in to comment.