Skip to content

Implement updated Deneb builder REST API#7665

Merged
StefanBratanov merged 5 commits intoConsensys:masterfrom
StefanBratanov:updated_deneb_builder_rest_api
Nov 6, 2023
Merged

Implement updated Deneb builder REST API#7665
StefanBratanov merged 5 commits intoConsensys:masterfrom
StefanBratanov:updated_deneb_builder_rest_api

Conversation

@StefanBratanov
Copy link
Copy Markdown
Contributor

@StefanBratanov StefanBratanov commented Nov 6, 2023

PR Description

Implement the required REST API changes as per ethereum/builder-specs#90

  • Changed BuilderBid to have blobKzgCommitments. Defaulted getOptionalBlindedBlobsBundle() to empty to avoid changing block production flow for now.
  • Removed the notion of SignedBlindedBlockContainer passing in getPayload. Passing only SignedBeaconBlock now.

Fixed Issue(s)

helps with builder block production flow for #7654

Documentation

  • I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

  • I thought about adding a changelog entry, and added one if I deemed necessary.

Copy link
Copy Markdown
Contributor

@zilm13 zilm13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@StefanBratanov StefanBratanov enabled auto-merge (squash) November 6, 2023 14:25
@StefanBratanov StefanBratanov force-pushed the updated_deneb_builder_rest_api branch from 1e54953 to 3894f97 Compare November 6, 2023 15:16
@StefanBratanov StefanBratanov merged commit 7940163 into Consensys:master Nov 6, 2023
@StefanBratanov StefanBratanov deleted the updated_deneb_builder_rest_api branch November 6, 2023 15:56
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.

2 participants