github/reth
1
1
Fork 0
mirror of https://github.com/paradigmxyz/reth.git synced 2026-01-23 14:18:05 -05:00
Code Issues Packages Projects Releases Wiki Activity

chore: reuse validation fn (#7911)

Browse Source
This commit is contained in:
Matthias Seitz
2024-04-26 13:14:27 +02:00
committed by GitHub
parent 688ee06e98
commit bb0809ce2e
1 changed files with 4 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
crates/consensus/beacon-core/src/lib.rs
View File

@@ -11,10 +11,10 @@
use reth_consensus::{Consensus, ConsensusError};
use reth_consensus_common::validation;
use reth_primitives::{
constants::MAXIMUM_EXTRA_DATA_SIZE, Chain, ChainSpec, Hardfork, Header, SealedBlock,
SealedHeader, EMPTY_OMMER_ROOT_HASH, U256,
Chain, ChainSpec, Hardfork, Header, SealedBlock, SealedHeader, EMPTY_OMMER_ROOT_HASH, U256,
};
use std::{sync::Arc, time::SystemTime};
/// Ethereum beacon consensus
///
/// This consensus engine does basic checks as outlined in the execution specs.
@@ -87,7 +87,7 @@ impl Consensus for BeaconConsensus {
// is greater than its parent timestamp.
// validate header extradata for all networks post merge
validate_header_extradata(header)?;
validation::validate_header_extradata(header)?;
// mixHash is used instead of difficulty inside EVM
// https://eips.ethereum.org/EIPS/eip-4399#using-mixhash-field-instead-of-difficulty
@@ -111,7 +111,7 @@ impl Consensus for BeaconConsensus {
// * If the network is goerli pre-merge, ignore the extradata check, since we do not
// support clique. Same goes for OP blocks below Bedrock.
if self.chain_spec.chain != Chain::goerli() && !self.chain_spec.is_optimism() {
validate_header_extradata(header)?;
validation::validate_header_extradata(header)?;
}
}
@@ -122,15 +122,3 @@ impl Consensus for BeaconConsensus {
validation::validate_block_standalone(block, &self.chain_spec)
}
}
/// Validates the header's extradata according to the beacon consensus rules.
///
/// From yellow paper: extraData: An arbitrary byte array containing data relevant to this block.
/// This must be 32 bytes or fewer; formally Hx.
fn validate_header_extradata(header: &Header) -> Result<(), ConsensusError> {
if header.extra_data.len() > MAXIMUM_EXTRA_DATA_SIZE {
Err(ConsensusError::ExtraDataExceedsMax { len: header.extra_data.len() })
} else {
Ok(())
}
}