Fix priority fees edge case

zeta-chain · Oct 1, 2024 · d1d209d · d1d209d
1 parent c05a91c
commit d1d209d
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 4 deletions.
diff --git a/zetaclient/chains/evm/signer/gas.go b/zetaclient/chains/evm/signer/gas.go
@@ -92,7 +92,12 @@ func gasFromCCTX(cctx *types.CrossChainTx, logger zerolog.Logger) (Gas, error) {
 	case err != nil:
 		return Gas{}, errors.Wrap(err, "unable to parse priorityFee")
 	case gasPrice.Cmp(priorityFee) == -1:
-		return Gas{}, fmt.Errorf("gasPrice (%d) is less than priorityFee (%d)", gasPrice.Int64(), priorityFee.Int64())
+		logger.Warn().
+			Uint64("cctx.initial_priority_fee", priorityFee.Uint64()).
+			Uint64("cctx.forced_priority_fee", gasPrice.Uint64()).
+			Msg("gasPrice is less than priorityFee, setting priorityFee = gasPrice")
+
+		priorityFee = gasPrice
 	}
 
 	return Gas{

diff --git a/zetaclient/chains/evm/signer/gas_test.go b/zetaclient/chains/evm/signer/gas_test.go
@@ -98,9 +98,16 @@ func TestGasFromCCTX(t *testing.T) {
 			errorContains: "unable to parse priorityFee: big.Int is negative",
 		},
 		{
-			name:          "gasPrice is less than priorityFee",
-			cctx:          makeCCTX(123_000, gwei(4).String(), gwei(5).String()),
-			errorContains: "gasPrice (4000000000) is less than priorityFee (5000000000)",
+			name: "gasPrice is less than priorityFee",
+			cctx: makeCCTX(123_000, gwei(4).String(), gwei(5).String()),
+			assert: func(t *testing.T, g Gas) {
+				assert.False(t, g.isLegacy())
+				assertGasEquals(t, Gas{
+					Limit:       123_000,
+					Price:       gwei(4),
+					PriorityFee: gwei(4),
+				}, g)
+			},
 		},
 		{
 			name:          "gasPrice is invalid",