fix: clear fees before retrying SLC

palomachain · Sep 5, 2024 · f8c0c3e · f8c0c3e
1 parent c3db1e1
commit f8c0c3e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/x/evm/keeper/attest_submit_logic_call.go b/x/evm/keeper/attest_submit_logic_call.go
@@ -65,6 +65,8 @@ func (a *submitLogicCallAttester) attemptRetry(ctx sdk.Context, proof *types.Sma
 	slc := a.action
 	if slc.Retries < cMaxSubmitLogicCallRetries {
 		slc.Retries++
+		// We must clear fees before retry or the signature verification fails
+		slc.Fees = nil
 		a.logger.Info("retrying failed SubmitLogicCall message",
 			"message-id", a.msgID,
 			"retries", slc.Retries,

diff --git a/x/evm/keeper/attest_upload_user_smart_contract.go b/x/evm/keeper/attest_upload_user_smart_contract.go
@@ -147,6 +147,8 @@ func (a *uploadUserSmartContractAttester) attemptRetry(ctx sdk.Context) {
 	}
 
 	a.action.Retries++
+	// We must clear fees before retry or the signature verification fails
+	a.action.Fees = nil
 
 	a.logger.Info("Retrying failed UploadUserSmartContract message",
 		"message-id", a.msgID,