We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
consensus, eips, genesis
currently blob validation is part of the 4844 tx:
alloy/crates/consensus/src/transaction/eip4844.rs
Lines 436 to 440 in 44b8a6d
it would be more useful to have the impl as a function on the sidecar:
pub fn validate( &self, versioned_hashes: &[B256], proof_settings: &KzgSettings, ) -> Result<(), BlobTransactionValidationError> {
this way it becomes reusable
the tx4844 impl then simply calls this will its versioned hashes and the given settings.
No response
The text was updated successfully, but these errors were encountered:
Successfully merging a pull request may close this issue.
Component
consensus, eips, genesis
Describe the feature you would like
currently blob validation is part of the 4844 tx:
alloy/crates/consensus/src/transaction/eip4844.rs
Lines 436 to 440 in 44b8a6d
it would be more useful to have the impl as a function on the sidecar:
this way it becomes reusable
the tx4844 impl then simply calls this will its versioned hashes and the given settings.
Additional context
No response
The text was updated successfully, but these errors were encountered: