Merge pull request #5309 from BitGo/COIN-2577-Optima-Financial-Error-when-transferring-USDC-Arbitrum-from-Multisig-Self-Custody-Wallet

venkateshv1266 · web-flow · commit 0ed8e3409206 · 2024-12-23T20:05:17.000+05:30
fix(abstract-eth): fix verifyCoin method to validate native coin
diff --git a/modules/abstract-eth/src/abstractEthLikeNewCoins.ts b/modules/abstract-eth/src/abstractEthLikeNewCoins.ts
@@ -2230,7 +2230,8 @@ export abstract class AbstractEthLikeNewCoins extends AbstractEthLikeCoin {
    * @returns {boolean}
    */
   verifyCoin(txPrebuild: TransactionPrebuild): boolean {
-    return txPrebuild.coin === this.getChain();
+    const nativeCoin = this.getChain().split(':')[0];
+    return txPrebuild.coin === nativeCoin;
   }
 
   /**

Original file line number	Diff line number	Diff line change
`@@ -2230,7 +2230,8 @@ export abstract class AbstractEthLikeNewCoins extends AbstractEthLikeCoin {`
`2230`	`2230`	`* @returns {boolean}`
`2231`	`2231`	`*/`
`2232`	`2232`	`verifyCoin(txPrebuild: TransactionPrebuild): boolean {`
`2233`		`- return txPrebuild.coin === this.getChain();`
	`2233`	`+ const nativeCoin = this.getChain().split(':')[0];`
	`2234`	`+ return txPrebuild.coin === nativeCoin;`
`2234`	`2235`	`}`
`2235`	`2236`
`2236`	`2237`	`/**`