Merge pull request #5322 from BitGo/COIN-2612-update-err-msg

chore: updated error message for gas limit and price

Author: venkateshv1266

modules/abstract-eth/src/abstractEthLikeNewCoins.ts

@@ -1373,7 +1373,7 @@ export abstract class AbstractEthLikeNewCoins extends AbstractEthLikeCoin {
       ? new optionalDeps.ethUtil.BN(params.eip1559.maxFeePerGas)
       : params.gasPrice
       ? new optionalDeps.ethUtil.BN(this.setGasPrice(params.gasPrice))
-      : await this.getGasPriceFromExternalAPI();
+      : await this.getGasPriceFromExternalAPI(this.staticsCoin?.name as string);
 
     const bitgoFeeAddressNonce = await this.getAddressNonce(bitgoFeeAddress);
 
@@ -1480,6 +1480,7 @@ export abstract class AbstractEthLikeNewCoins extends AbstractEthLikeCoin {
     if (!params.gasLimit && userKey && !userKey.startsWith('xpub')) {
       const sendData = txBuilder.getSendData();
       gasLimit = await this.getGasLimitFromExternalAPI(
+        params.intendedChain as string,
         params.bitgoFeeAddress as string,
         params.walletContractAddress,
         sendData
@@ -1650,6 +1651,7 @@ export abstract class AbstractEthLikeNewCoins extends AbstractEthLikeCoin {
     if (!params.gasLimit && userKey && !userKey.startsWith('xpub')) {
       const sendData = txBuilder.getSendData();
       gasLimit = await this.getGasLimitFromExternalAPI(
+        params.intendedChain as string,
         params.bitgoFeeAddress as string,
         params.walletContractAddress,
         sendData
@@ -2447,7 +2449,7 @@ export abstract class AbstractEthLikeNewCoins extends AbstractEthLikeCoin {
   /**
    * Fetch the gas price from the explorer
    */
-  async getGasPriceFromExternalAPI(): Promise<BN> {
+  async getGasPriceFromExternalAPI(wrongChainCoin: string): Promise<BN> {
     try {
       const res = await this.recoveryBlockchainExplorerQuery({
         module: 'proxy',
@@ -2457,17 +2459,18 @@ export abstract class AbstractEthLikeNewCoins extends AbstractEthLikeCoin {
       console.log(` Got gas price: ${gasPrice}`);
       return gasPrice;
     } catch (e) {
-      throw new Error('Failed to get gas price');
+      throw new Error(`Failed to get gas price. Please make sure to use the api key of ${wrongChainCoin}`);
     }
   }
 
   /**
    * Fetch the gas limit from the explorer
+   * @param intendedChain
    * @param from
    * @param to
    * @param data
    */
-  async getGasLimitFromExternalAPI(from: string, to: string, data: string): Promise<BN> {
+  async getGasLimitFromExternalAPI(intendedChain: string, from: string, to: string, data: string): Promise<BN> {
     try {
       const res = await this.recoveryBlockchainExplorerQuery({
         module: 'proxy',
@@ -2480,7 +2483,9 @@ export abstract class AbstractEthLikeNewCoins extends AbstractEthLikeCoin {
       console.log(`Got gas limit: ${gasLimit}`);
       return gasLimit;
     } catch (e) {
-      throw new Error('Failed to get gas limit: ');
+      throw new Error(
+        `Failed to get gas limit. Please make sure to use the privateKey aka userKey of ${intendedChain} wallet ${to}`
+      );
     }
   }
 

modules/sdk-coin-avaxc/src/avaxc.ts

@@ -1212,11 +1212,12 @@ export class AvaxC extends AbstractEthLikeNewCoins {
 
   /**
    * Fetch the gas limit from the explorer
+   * @param intendedChain
    * @param from
    * @param to
    * @param data
    */
-  async getGasLimitFromExternalAPI(from: string, to: string, data: string): Promise<BN> {
+  async getGasLimitFromExternalAPI(intendedChain: string, from: string, to: string, data: string): Promise<BN> {
     try {
       const res = await this.recoveryBlockchainExplorerQuery({
         jsonrpc: '2.0',
@@ -1235,7 +1236,9 @@ export class AvaxC extends AbstractEthLikeNewCoins {
       console.log(`Got gas limit: ${gasLimit}`);
       return gasLimit;
     } catch (e) {
-      throw new Error('Failed to get gas limit: ');
+      throw new Error(
+        `Failed to get gas limit. Please make sure to use the privateKey aka userKey of ${intendedChain} wallet ${to}`
+      );
     }
   }
 }