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

DepositSnapshotResponse.finalized is incorrectly set to exactly three items #424

Closed prestonvanloon closed 7 months ago

prestonvanloon commented 7 months ago

Looking at this API response object.

https://github.com/ethereum/beacon-APIs/blob/d2f846e0e0d551973253df4b5341655eddf92f1d/types/api.yaml#L3-L20

However, the EIP spec mentions the following:

Where finalized is a variable-length list (of maximum size DEPOSIT_CONTRACT_DEPTH) containing the hashes defined in the Deposit Finalization Flow section

Therefore the API definitions should be corrected:

cc @ethDreamer