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

refactor(light-client): Replace ssz-rs with ethereum_ssz in Light Client #1588

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

virajbhartiya
Copy link

Closes #1417

What was wrong?

The light-client module was using ssz-rs for SSZ handling, while the rest of the project primarily relied on ethereum_ssz. This inconsistency introduced an additional dependency and potential maintainability challenges.

How was it fixed?

Replaced ssz-rs with ethereum_ssz in the light-client module. Adjusted usages of types like Vector and Node to their equivalents in ethereum_ssz (FixedVector and B256, respectively). Updated the relevant code paths to align with the new library's API.

@virajbhartiya
Copy link
Author

I was able to replace the usage of ssz-rs for Vector and Node, running into a few errors for replacement of hash_tree_root and is_valid_merkle_branch

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.

In light-client, use ethereum_ssz instead of ssz-rs
1 participant