-
Notifications
You must be signed in to change notification settings - Fork 201
added pending deposits and pending partial withdrawals accessors #500
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
Conversation
This reverts commit 4cc8484.
|
that log is wild, but the diff looks right... |
If all server follow the defined behavior here it's possible to add SSZ support later on as it's backward compatible due to proper content type negotiation between server and client. But I am definitely in favor of adding SSZ support to new apis |
Partially addresses ethereum/beacon-APIs#500 Attempting to access this data prior to electra will fail with a 400. Signed-off-by: Paul Harris <[email protected]>
Partially addresses ethereum/beacon-APIs#500 Attempting to access this data prior to electra will fail with a 400. Signed-off-by: Paul Harris <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally fine with adding those apis as they are pretty simple to implement. Left a few comments which are applicable to both endpoints.
Co-authored-by: Nico Flaig <[email protected]>
Partially addresses ethereum/beacon-APIs#500 Attempting to access this data prior to electra will fail with a 400. Signed-off-by: Paul Harris <[email protected]>
Partially addresses ethereum/beacon-APIs#500 Attempting to access this data prior to electra will fail with a 400. Signed-off-by: Paul Harris <[email protected]>
…als (#7472) **Motivation** - ethereum/beacon-APIs#500 **Description** - `GET /eth/v1/beacon/states/{state_id}/pending_deposits` added - `GET /eth/v1/beacon/states/{state_id}/pending_partial_withdrawals` added
* 25.2.0 Cleanup (#9109) * update electra fork epoch (#9093) * update electra epoch version Signed-off-by: gconnect <[email protected]> * Remove MAX_REQUEST_BLOB_SIDECARS Signed-off-by: gconnect <[email protected]> * Add MAX_REQUEST_BLOB_SIDECARS Signed-off-by: gconnect <[email protected]> --------- Signed-off-by: gconnect <[email protected]> Co-authored-by: Enrico Del Fante <[email protected]> * Add `SingleAttestation` in pretty print (#9101) * Add SingleAttestation in pretty print * Add SingleAttestation in pretty print * Added GetPendingDeposits (#9110) Partially addresses ethereum/beacon-APIs#500 Attempting to access this data prior to electra will fail with a 400. Signed-off-by: Paul Harris <[email protected]> * upgrade besu library (#9116) Signed-off-by: Gabriel Fukushima <[email protected]> * added getPendingPartialWithdrawals api for electra (#9113) Partially addresses ethereum/beacon-APIs#500 Attempting to access this data prior to electra will fail with a 400. Signed-off-by: Paul Harris <[email protected]> * Improve attestation selection (#9114) * improve attestation selection * add test * improvements * no need for entrySet * rename var --------- Co-authored-by: Paul Harris <[email protected]> * fix mev gaslimit warning (#9136) * builder-json-parsing-fix (#9135) * set Accept header in getPayload (#9137) --------- Signed-off-by: gconnect <[email protected]> Signed-off-by: Paul Harris <[email protected]> Signed-off-by: Gabriel Fukushima <[email protected]> Co-authored-by: Lucas Saldanha <[email protected]> Co-authored-by: Glory Agatevure <[email protected]> Co-authored-by: Enrico Del Fante <[email protected]> Co-authored-by: Paul Harris <[email protected]> Co-authored-by: Gabriel Fukushima <[email protected]>
…g_partial_withdrawals) (#7006) Resolves #7003 Added two endpoints as ethereum/beacon-APIs#500 proposed: - `/eth/v1/beacon/states/{state_id}/pending_deposits` - `/eth/v1/beacon/states/{state_id}/pending_partial_withdrawals`
There was interest expressed in having access to pending deposits and pending partial withdrawals without needing to download the entire beacon state, and that makes a lot of sense.
I haven't added SSZ support to these at this point, but arguably we probably could...