-
Notifications
You must be signed in to change notification settings - Fork 386
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
engine: Make execution requests a sidecar, take 2 (#591)
* engine: extract execution requests from payload * add executionrequestsv to wordlist * update engine_getPayloadV4 * Make execution requests a sidecar, take 2 * Turn executionRequests into a sequence of bytes * Simplify wording * Clean up wordlist * Applied suggestions by @marioevz Co-authored-by: Mario Vega <[email protected]> * Switch to the nested list representation * Fix typo * Apply suggestions from @lucassaldanha Co-authored-by: Lucas Saldanha <[email protected]> * Switch getPayloadV4 response to ExecutionPayloadV3 * Replace hash with full executionRequests object for newPayloadV4 * Fix the newPayloadV4 note * Mention that requestType byte isn't part of encoding * Mention SSZ encoding in the executionRequests list * Apply suggestions from @lucassaldanha Co-authored-by: Lucas Saldanha <[email protected]> * Update payload.yaml examples * Drop 32 bytes len from executionRequests definition * engine: clarification on requests --------- Co-authored-by: Roman Krasiuk <[email protected]> Co-authored-by: Mario Vega <[email protected]> Co-authored-by: Lucas Saldanha <[email protected]> Co-authored-by: lightclient <[email protected]>
- Loading branch information
1 parent
ad9102b
commit 4140e52
Showing
4 changed files
with
36 additions
and
186 deletions.
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -68,4 +68,3 @@ src | |
https | ||
forkchoiceupdatedresponsev | ||
exitv | ||
depositreceiptv |