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

feat(arbitrum): light client #1868

Merged
merged 1 commit into from
May 9, 2024
Merged

feat(arbitrum): light client #1868

merged 1 commit into from
May 9, 2024

Conversation

benluelo
Copy link
Contributor

@benluelo benluelo commented May 3, 2024

  • add Arbitrum to chain-utils
  • add Arbitrum to block-message
  • add Arbitrum to relay-message
  • create proto definitions for light client
  • arbitrum-verifier
    • untested, will add testdata in followup pr
  • arbitrum-light-client
  • docs on how the light client and state tracking works
    • do NOT let me merge this PR without this one checked off. I promise you I will try
    • this is documented in arbitrum.proto

Other misc. changes made in this PR:

  • implement IntoIterator<Item = u8> for byte array wrapper types directly
  • tracing improvements
  • a few ICEs for good measure
  • start exposing useful fetching actions via the cli (query latest height) both generically and for specific chains
  • add chain id to all voyager logs (where applicable)
  • clean up error types in ethereum-verifier and arbitrum-verifier (print full error stack since cosmwasm doesn't call .source() on the error type, it just uses Display)
  • devnet-union proposal period is now 30 seconds, not 2 days

@benluelo benluelo force-pushed the voyager-arbitrum branch from 66de117 to 22cc1c2 Compare May 8, 2024 19:34
@benluelo benluelo changed the title add arbitrum support feat(arbitrum): light client May 8, 2024
@benluelo benluelo marked this pull request as ready for review May 8, 2024 19:35
@benluelo benluelo force-pushed the voyager-arbitrum branch from 22cc1c2 to 8adb6dd Compare May 8, 2024 19:40
@benluelo benluelo merged commit 5a3e331 into main May 9, 2024
36 checks passed
@benluelo benluelo deleted the voyager-arbitrum branch May 9, 2024 03:21
hussein-aitlahcen added a commit that referenced this pull request May 9, 2024
these were left as stubs in #1868 as i hadn't tested packets yet.
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.

2 participants