Skip to content

Commit

Permalink
Suppress noisy chainID logs
Browse files Browse the repository at this point in the history
  • Loading branch information
yzang2019 committed Oct 8, 2024
1 parent 4daa00c commit 5c2afa8
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions x/evm/ante/sig.go
Original file line number Diff line number Diff line change
Expand Up @@ -47,13 +47,13 @@ func (svd *EVMSigVerifyDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, simulat
case ethtypes.LegacyTxType:
// legacy either can have a zero or correct chain ID
if txChainID.Cmp(big.NewInt(0)) != 0 && txChainID.Cmp(chainID) != 0 {
ctx.Logger().Error("chainID mismatch", "txChainID", ethTx.ChainId(), "chainID", chainID)
ctx.Logger().Debug("chainID mismatch", "txChainID", ethTx.ChainId(), "chainID", chainID)

Check warning on line 50 in x/evm/ante/sig.go

View check run for this annotation

Codecov / codecov/patch

x/evm/ante/sig.go#L50

Added line #L50 was not covered by tests
return ctx, sdkerrors.ErrInvalidChainID
}
default:
// after legacy, all transactions must have the correct chain ID
if txChainID.Cmp(chainID) != 0 {
ctx.Logger().Error("chainID mismatch", "txChainID", ethTx.ChainId(), "chainID", chainID)
ctx.Logger().Debug("chainID mismatch", "txChainID", ethTx.ChainId(), "chainID", chainID)

Check warning on line 56 in x/evm/ante/sig.go

View check run for this annotation

Codecov / codecov/patch

x/evm/ante/sig.go#L56

Added line #L56 was not covered by tests
return ctx, sdkerrors.ErrInvalidChainID
}
}
Expand Down

0 comments on commit 5c2afa8

Please sign in to comment.