Skip to content

engine: add POST /{fork}/payloads/witness to the REST+SSZ refactor#1

Open
developeruche wants to merge 2 commits into
MariusVanDerWijden:rest-sszfrom
developeruche:feat/rest-ssz-payloads-witness
Open

engine: add POST /{fork}/payloads/witness to the REST+SSZ refactor#1
developeruche wants to merge 2 commits into
MariusVanDerWijden:rest-sszfrom
developeruche:feat/rest-ssz-payloads-witness

Conversation

@developeruche

Copy link
Copy Markdown

What this does

Adds the witness route to the REST + SSZ Engine API refactor:

POST /engine/v2/{fork}/payloads/witness: same request as POST /{fork}/payloads, but the response is a superset of PayloadStatus that also carries the stateless ExecutionWitness. It folds the legacy two-call flow (engine_newPayloaddebug_executionWitness) into a single round-trip.

This integrates the proposal from ethereum#773 into this draft, per the conclusion reached on the ACD call and in the threads below. It replaces ethereum#773's standalone POST /new-payload-with-witness endpoint with one that follows this spec's resource model and SSZ conventions.

Changes

refactor.md

  • New "Payload submission with witness" section defining the endpoint
  • Rows added to the old → new mapping table and the resource model table
  • TOC entry

refactor-ssz.md

  • ExecutionWitness shared container (state / codes / headers as ByteLists)
  • PayloadStatusWithWitness response container for the endpoint
  • MAX_WITNESS_ITEMS / MAX_WITNESS_ITEM_BYTES constants
  • TOC entries

Related

cc @MariusVanDerWijden @LukaszRozmej @RazorClient

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant