Skip to content

Conversation

@djrtwo
Copy link
Contributor

@djrtwo djrtwo commented Mar 15, 2023

mkalinin and others added 30 commits December 24, 2022 09:40
(This was probably a cut-n-paste from block validation)
The spec currently defines `MAX_REQUEST_BLOB_SIDECARS` as the "maximum
number of blob sidecars in a single request", but then later in the
RPC description defines the max is `MAX_REQUEST_BLOB_SIDECARS *
MAX_BLOBS_PER_BLOCK`.

Clean this up by defining `MAX_REQUEST_BLOB_SIDECARS` to be the actual max.
Fix outdated (likely a Bellatrix cut-paste) description of change.
Minor typo/pasta fixes in blob_sidecar_{index} validation
Remove the outdated description in Deneb fork
@djrtwo djrtwo closed this Mar 16, 2023
@djrtwo djrtwo reopened this Mar 16, 2023
Copy link
Contributor

@hwwhww hwwhww left a comment

Choose a reason for hiding this comment

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

test vectors are ready :shipit:

@djrtwo djrtwo changed the title [do not merge] release v1.3.0-rc.4 Release v1.3.0-rc.4 Mar 16, 2023
@djrtwo djrtwo merged commit df8ff7f into master Mar 16, 2023
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.