Update attestation APIs to support EIP-7549

ethereum / beacon-APIs

Collection of RESTful APIs provided by Ethereum Beacon nodes

https://ethereum.github.io/beacon-APIs/

Creative Commons Zero v1.0 Universal

328 stars 167 forks source link

Update attestation APIs to support EIP-7549 #448

Closed mehdi-aouadi closed 2 months ago

mehdi-aouadi commented 3 months ago

Following the Electra containers updates:

Update the /block and /pool attestation GET APIs to handle the new Electra attestations
Add Electra.Attestation schema
Extract the AttestationData schema used by both Attestation and Electra.Attestation
Update the attester slashing API
Update the IndexedAttestation schema

Partially fixes https://github.com/ethereum/beacon-APIs/issues/445 Related to https://github.com/ethereum/beacon-APIs/pull/447

nflaig commented 3 months ago

What about attestation event, should it be part of this PR?

https://github.com/ethereum/beacon-APIs/blob/fb718b65d39b62ffd4ee6c95d1d6fb1458c08958/apis/eventstream/index.yaml#L64-L68

mehdi-aouadi commented 3 months ago

What about attestation event, should it be part of this PR?

https://github.com/ethereum/beacon-APIs/blob/fb718b65d39b62ffd4ee6c95d1d6fb1458c08958/apis/eventstream/index.yaml#L64-L68

I considered the event stream events out of the scope of this PR. Might be addressed in a separate PR imo

rolfyone commented 3 months ago

Yep I agree with new version for interfaces given they're a new required field (breaking) and headers. minor comment about references to /dev versions.