Make occ caused evm panics less noisy (#1728)

* make occ caused evm panics less noisy * update comment * remove prints * fix * fix
sei-protocol · Jun 11, 2024 · 6353441 · 6353441
1 parent 033cf29
commit 6353441
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/x/evm/keeper/msg_server.go b/x/evm/keeper/msg_server.go
@@ -8,10 +8,12 @@ import (
 	"math"
 	"math/big"
 	"runtime/debug"
+	"strings"
 
 	"github.com/armon/go-metrics"
 	"github.com/cosmos/cosmos-sdk/telemetry"
 	sdk "github.com/cosmos/cosmos-sdk/types"
+	occtypes "github.com/cosmos/cosmos-sdk/types/occ"
 	bankkeeper "github.com/cosmos/cosmos-sdk/x/bank/keeper"
 	banktypes "github.com/cosmos/cosmos-sdk/x/bank/types"
 	"github.com/ethereum/go-ethereum/common"
@@ -62,10 +64,11 @@ func (server msgServer) EVMTransaction(goCtx context.Context, msg *types.MsgEVMT
 
 	defer func() {
 		if pe := recover(); pe != nil {
-			// there is not supposed to be any panic
-			debug.PrintStack()
-			ctx.Logger().Error(fmt.Sprintf("EVM PANIC: %s", pe))
-			telemetry.IncrCounter(1, types.ModuleName, "panics")
+			if !strings.Contains(fmt.Sprintf("%s", pe), occtypes.ErrReadEstimate.Error()) {
+				debug.PrintStack()
+				ctx.Logger().Error(fmt.Sprintf("EVM PANIC: %s", pe))
+				telemetry.IncrCounter(1, types.ModuleName, "panics")
+			}
 			server.AppendErrorToEvmTxDeferredInfo(ctx, tx.Hash(), fmt.Sprintf("%s", pe))
 
 			panic(pe)