From 377f19ff4a2818c0866dd5afc7b252ad1172a776 Mon Sep 17 00:00:00 2001 From: "Masih H. Derkani" Date: Thu, 28 Aug 2025 09:13:36 +0100 Subject: [PATCH] Remove duplicate panic recovery in process proposal The panic recovery block seems to be repeated twice. This might have been an error rebasing or found its way in by accident? --- baseapp/abci.go | 14 -------------- 1 file changed, 14 deletions(-) diff --git a/baseapp/abci.go b/baseapp/abci.go index a266c94e..f45a9047 100644 --- a/baseapp/abci.go +++ b/baseapp/abci.go @@ -1117,20 +1117,6 @@ func (app *BaseApp) ProcessProposal(ctx context.Context, req *abci.RequestProces } }() - defer func() { - if err := recover(); err != nil { - app.logger.Error( - "panic recovered in ProcessProposal", - "height", req.Height, - "time", req.Time, - "hash", fmt.Sprintf("%X", req.Hash), - "panic", err, - ) - - resp = &abci.ResponseProcessProposal{Status: abci.ResponseProcessProposal_REJECT} - } - }() - if app.processProposalHandler != nil { resp, err = app.processProposalHandler(app.processProposalState.ctx, req) if err != nil {