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

Hyperlane Teller #34

Closed
junkim012 opened this issue Dec 5, 2024 · 0 comments · May be fixed by #35
Closed

Hyperlane Teller #34

junkim012 opened this issue Dec 5, 2024 · 0 comments · May be fixed by #35
Assignees

Comments

@junkim012
Copy link
Contributor

junkim012 commented Dec 5, 2024

Goal

  • Create a Teller that can use hyperlane as its underlying messaging protocol.

Requirements

  • Bridge Security
    • The destination chain teller should receive the message only from its counterpart teller on the source chain.
    • Only the mailbox should be able to call handle.
    • Users should be refunded overpaid gas for bridging.
    • When users define gas limit, this is in the destination chain's token, not the source chain's token.
  • Test
    • Each message must have its own guid.
    • Perform bridging to and from testnets.
  • Deployment
    • HyperlaneTeller deployment scripts
    • HyperlaneTeller configuration files

Discussions

Hyperlane Configurations

  • Using default ISMs vs. custom ISMs.
  • Gas refund address override
  • gas limit override

Slow Path vs. Fast Path (Not Implemented)

  • Slow Path (bus)
    • User can queue a bridging message for it to be processed later.
  • Fast Path (taxi)
    • User initiates an instant bridging message.
    • When the queue is non-empty, should the fast path initiator process the queue? Or should the fast path initiator not be responsible for paying for the looping costs of multiple vault.enters.
  • How to process multiple requests.
    • The queue data—an array of share amounts and recipients—can be sent over in one transaction, and the receiving leg needs to loop and call multiple vault.enters.
@junkim012 junkim012 self-assigned this Dec 5, 2024
@junkim012 junkim012 linked a pull request Dec 6, 2024 that will close this issue
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 a pull request may close this issue.

1 participant