Skip to content

Commit 521f0fd

Browse files
authored
Pass aggregate.committee_bits to get_committee_indices (#4742)
Correct the beacon_aggregate_and_proof validation to call get_committee_indices(aggregate.committee_bits) instead of get_committee_indices(aggregate). Electra defines get_committee_indices to accept a committee bitvector and all related code paths and tests use .committee_bits. This change aligns the networking spec with the Electra attestation format introduced by EIP-7549 and avoids implementer confusion.
1 parent c656bc4 commit 521f0fd

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

specs/electra/p2p-interface.md

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -98,14 +98,16 @@ The derivation of the `message-id` remains stable.
9898

9999
###### `beacon_aggregate_and_proof`
100100

101-
The following convenience variables are re-defined
101+
Assuming the alias `aggregate = signed_aggregate_and_proof.message.aggregate`:
102+
103+
The following convenience variables are re-defined:
102104

103105
- `index = get_committee_indices(aggregate.committee_bits)[0]`
104106

105107
The following validations are added:
106108

107109
- [REJECT] `len(committee_indices) == 1`, where
108-
`committee_indices = get_committee_indices(aggregate)`.
110+
`committee_indices = get_committee_indices(aggregate.committee_bits)`.
109111
- [REJECT] `aggregate.data.index == 0`
110112

111113
###### `blob_sidecar_{subnet_id}`

0 commit comments

Comments
 (0)