feat(price-pusher): add gas multiplier cap for evm

pyth-network · Oct 10, 2023 · fe070ba · fe070ba
1 parent f224486
commit fe070ba
Show file tree

Hide file tree

Showing 4 changed files with 21 additions and 4 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/price_pusher/package.json b/price_pusher/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@pythnetwork/price-pusher",
-  "version": "5.4.11",
+  "version": "5.5.0",
   "description": "Pyth Price Pusher",
   "homepage": "https://pyth.network",
   "main": "lib/index.js",

diff --git a/price_pusher/src/evm/command.ts b/price_pusher/src/evm/command.ts
@@ -36,11 +36,22 @@ export default {
     } as Options,
     "override-gas-price-multiplier": {
       description:
-        "Multiply the gas price by this number if the transaction is not landing to override it. Default to 1.1",
+        "Multiply the previous gas price by this number if the transaction is not landing to override. " +
+        "Please note that the gas price can grow exponentially on consecutive failures; " +
+        "to set a cap on the multiplier, use the `override-gas-price-multiplier-cap` option." +
+        "Default to 1.1",
       type: "number",
       required: false,
       default: 1.1,
     } as Options,
+    "override-gas-price-multiplier-cap": {
+      description:
+        "Maximum gas price multiplier to use in override compared to the RPC returned " +
+        "gas price. Default to 5",
+      type: "number",
+      required: false,
+      default: 5,
+    } as Options,
     ...options.priceConfigFile,
     ...options.priceServiceEndpoint,
     ...options.mnemonicFile,
@@ -61,6 +72,7 @@ export default {
       customGasStation,
       txSpeed,
       overrideGasPriceMultiplier,
+      overrideGasPriceMultiplierCap,
     } = argv;
 
     const priceConfigs = readPriceConfigFile(priceConfigFile);
@@ -106,6 +118,7 @@ export default {
       priceServiceConnection,
       pythContractFactory,
       overrideGasPriceMultiplier,
+      overrideGasPriceMultiplierCap,
       gasStation
     );
 

diff --git a/price_pusher/src/evm/evm.ts b/price_pusher/src/evm/evm.ts
@@ -129,6 +129,7 @@ export class EvmPricePusher implements IPricePusher {
     private connection: PriceServiceConnection,
     pythContractFactory: PythContractFactory,
     private overrideGasPriceMultiplier: number,
+    private overrideGasPriceMultiplierCap: number,
     customGasStation?: CustomGasStation
   ) {
     this.customGasStation = customGasStation;
@@ -200,7 +201,10 @@ export class EvmPricePusher implements IPricePusher {
     }
 
     if (gasPriceToOverride !== undefined && gasPriceToOverride > gasPrice) {
-      gasPrice = gasPriceToOverride;
+      gasPrice = Math.min(
+        gasPriceToOverride,
+        gasPrice * this.overrideGasPriceMultiplierCap
+      );
     }
 
     const txNonce = lastExecutedNonce + 1;