Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Warp Route Wrapper Contract for Hyperlane #31

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

junkim012
Copy link
Contributor

@junkim012 junkim012 commented Oct 23, 2024

  • Because hyperlane does not use the CrossChainTellerBase burn and mint bridge, it needs to be a separate transferRemote transaction after calling deposit.
  • This causes UX issues such as deposit call succeeding but the transferRemote call failing, forcing users to bridge manually from the source chain to the destination chain.
  • This is a simple contract that allows both calls to be made atomically.
  • Security wise, this contract is only meant to be a pass through function, and does not custody any assets before or after the call.

src/helper/WarpRouteWrapper.sol Show resolved Hide resolved
src/helper/WarpRouteWrapper.sol Outdated Show resolved Hide resolved
src/helper/WarpRouteWrapper.sol Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants