Skip to content

Commit

Permalink
fix: deny stargateV2Bus in lifi swapper (#8003)
Browse files Browse the repository at this point in the history
  • Loading branch information
woodenfurniture authored Oct 25, 2024
1 parent 0dbfa73 commit ab0c7c0
Showing 1 changed file with 7 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,13 @@ export async function getTradeQuote(
// used for analytics and affiliate fee - do not change this without considering impact
integrator: LIFI_INTEGRATOR_ID,
slippage: Number(slippageTolerancePercentageDecimal),
bridges: { deny: ['stargate', 'stargateV2', 'amarok', 'arbitrum'] },
// Routes via Stargate or Amarok can always be executed in one step, as LiFi can make those
// bridges swap into any requested token on the destination chain. Other bridges my require
// two steps. As such, additional balance checks on the destination chain are required which
// are currently incompatible with our fee calculations, leading to incorrect fee display,
// reverts, partial swaps, wrong received tokens (due to out-of-gas mid-trade), etc. For now,
// these bridges are disabled.
bridges: { deny: ['stargate', 'stargateV2', 'stargateV2Bus', 'amarok', 'arbitrum'] },
allowSwitchChain: true,
fee: affiliateBpsDecimalPercentage.isZero()
? undefined
Expand Down

0 comments on commit ab0c7c0

Please sign in to comment.