From 43083dfd49639e584d830e9d86830e531fdc052a Mon Sep 17 00:00:00 2001 From: Luke Parker Date: Sun, 21 Apr 2024 05:32:41 -0400 Subject: [PATCH] Remove redundant log from tendermint lib --- coordinator/tributary/tendermint/src/lib.rs | 16 ++-------------- 1 file changed, 2 insertions(+), 14 deletions(-) diff --git a/coordinator/tributary/tendermint/src/lib.rs b/coordinator/tributary/tendermint/src/lib.rs index edd26183..9ee71a9d 100644 --- a/coordinator/tributary/tendermint/src/lib.rs +++ b/coordinator/tributary/tendermint/src/lib.rs @@ -679,18 +679,6 @@ impl TendermintMachine { Err(TendermintError::Temporal)?; } - if (msg.block == self.block.number) && - (msg.round == self.block.round().number) && - (msg.data.step() == Step::Propose) - { - log::trace!( - target: "tendermint", - "received Propose for block {}, round {}", - msg.block.0, - msg.round.0, - ); - } - // If this is a precommit, verify its signature self.verify_precommit_signature(signed).await?; @@ -698,7 +686,7 @@ impl TendermintMachine { if matches!(msg.data, Data::Proposal(..)) && (msg.sender != self.weights.proposer(msg.block, msg.round)) { - log::warn!(target: "tendermint", "Validator who wasn't the proposer proposed"); + log::warn!(target: "tendermint", "validator who wasn't the proposer proposed"); // TODO: This should have evidence self .slash(msg.sender, SlashEvent::Id(SlashReason::InvalidProposer, msg.block.0, msg.round.0)) @@ -971,7 +959,7 @@ impl TendermintMachine { // Only run if it's still the step in question if self.block.round().step == step { // Slash the validator for not proposing when they should've - log::debug!(target: "tendermint", "Validator didn't propose when they should have"); + log::debug!(target: "tendermint", "validator didn't propose when they should have"); // this slash will be voted on. self.slash( self.weights.proposer(self.block.number, self.block.round().number),