fix: dedup bridges to approve properly (#1908)

* fix: dedup bridges to approve properly Signed-off-by: bennett <[email protected]> --------- Signed-off-by: bennett <[email protected]>
across-protocol · Nov 13, 2024 · f644828 · f644828
1 parent 28cffcd
commit f644828
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/adapter/BaseChainAdapter.ts b/src/adapter/BaseChainAdapter.ts
@@ -148,8 +148,12 @@ export class BaseChainAdapter {
       ),
     ]);
     // Dedup the `gasTokensToApprove` array so that we don't approve the same bridge to send the same token multiple times.
+    const tokenBridgePairs = gasTokensToApprove.map(({ token, bridges }) => `${token.address}_${bridges.join("_")}`);
     const tokensToApprove = gasTokensToApprove
-      .filter(({ token, bridges }, idx) => gasTokensToApprove.indexOf({ token, bridges }) === idx)
+      .filter(({ token, bridges }, idx) => {
+        const tokenBridgePair = `${token.address}_${bridges.join("_")}`;
+        return tokenBridgePairs.indexOf(tokenBridgePair) === idx;
+      })
       .concat(bridgeTokensToApprove)
       .filter(({ bridges }) => bridges.length > 0);
     if (unavailableTokens.length > 0) {