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: implement DynSSZ support #194

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

filoozom
Copy link

@filoozom filoozom commented Mar 6, 2025

Implements support for DynSSZ thanks to attestantio/go-eth2-client#123.

This is very useful for non-standard networks, like devnets, testnets or specifically Gnosis Chain and Chiado in our case. This way, all the configuration variables are fetched automatically from the nodes and used to decode data and calculate roots.

I'm not a Go developer, so this is probably not the right way to do things. However, it does work and demonstrate its usefulness. A few issues for sure:

  • Only fetching the data from one node instead of using them all
  • Fetching the spec on every calculation instead of caching it
  • Error management

I'm hoping that someone can help me finish this in an idiomatic way!

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.

1 participant