Skip to content

Add BlobAndProofV2 #2201

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

Closed
Rjected opened this issue Mar 14, 2025 · 1 comment · Fixed by #2202
Closed

Add BlobAndProofV2 #2201

Rjected opened this issue Mar 14, 2025 · 1 comment · Fixed by #2202
Assignees

Comments

@Rjected
Copy link
Contributor

Rjected commented Mar 14, 2025

Right now we have BlobAndProofV1:

//! Misc types related to the 4844
use crate::eip4844::{Blob, Bytes48};
use alloc::boxed::Box;
/// Blob type returned in responses to `engine_getBlobsV1`: <https://github.com/ethereum/execution-apis/pull/559>
#[derive(Debug, Clone, PartialEq, Eq)]
#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
pub struct BlobAndProofV1 {
/// The blob data.
pub blob: Box<Blob>,
/// The KZG proof for the blob.
pub proof: Bytes48,
}

A struct exactly like this should be added, called BlobAndProofV2, which contains the same fields as BlobAndProofV1 but with cell_proof instead of proof. The serde struct cfg will have to be adjusted to use camel case on all fields as well.

This link should be included in the docs as well:
ethereum/execution-apis#630

@varun-doshi
Copy link
Contributor

I'd like to work on this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants