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

Make sync committee type conform to spec #160

Merged
merged 1 commit into from
Aug 17, 2021

Conversation

rkapka
Copy link
Collaborator

@rkapka rkapka commented Aug 11, 2021

The altair response schema for https://ethereum.github.io/eth2.0-APIs/#/Debug/getStateV2 fields current_sync_committee and next_sync_committee are not in line with the spec. pubkeys length is 1024 and pubkey_aggregates is an array, whereas https://github.com/ethereum/eth2.0-specs/blob/dev/specs/altair/beacon-chain.md#synccommittee specifies pubkeys of length 512 (SYNC_COMMITTEE_SIZE) and an aggregate_pubkey field which is a single public key.

Copy link
Collaborator

@mpetrunic mpetrunic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for late review, vacations and stuff. Tnx for fixing that, gonna wait a bit if we encounter some other bug before doing some patch release

@mpetrunic mpetrunic merged commit 6be3cee into ethereum:master Aug 17, 2021
@rkapka rkapka deleted the fix-altair-state branch March 23, 2022 11:42
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