fix: call validate_execution_requests from validate_version_specific_fields for ethereum engine validator

crates/ethereum/node/src/engine.rs

@@ -9,8 +9,8 @@ use reth_chainspec::ChainSpec;
 use reth_engine_primitives::{EngineTypes, EngineValidator, PayloadValidator};
 use reth_ethereum_payload_builder::EthereumExecutionPayloadValidator;
 use reth_payload_primitives::{
-    validate_version_specific_fields, EngineApiMessageVersion, EngineObjectValidationError,
-    NewPayloadError, PayloadOrAttributes,
+    validate_execution_requests, validate_version_specific_fields, EngineApiMessageVersion,
+    EngineObjectValidationError, NewPayloadError, PayloadOrAttributes,
 };
 use reth_primitives::{Block, RecoveredBlock};
 use std::sync::Arc;
@@ -56,6 +56,11 @@ where
         version: EngineApiMessageVersion,
         payload_or_attrs: PayloadOrAttributes<'_, Self::ExecutionData, EthPayloadAttributes>,
     ) -> Result<(), EngineObjectValidationError> {
+        payload_or_attrs
+            .execution_requests()
+            .map(|requests| validate_execution_requests(requests))
+            .transpose()?;
+
         validate_version_specific_fields(self.chain_spec(), version, payload_or_attrs)
     }
 

crates/payload/primitives/src/payload.rs

@@ -1,6 +1,9 @@
 use crate::{MessageValidationKind, PayloadAttributes};
 use alloc::vec::Vec;
-use alloy_eips::eip4895::Withdrawal;
+use alloy_eips::{
+    eip4895::Withdrawal,
+    eip7685::{Requests, RequestsOrHash},
+};
 use alloy_primitives::B256;
 use alloy_rpc_types_engine::ExecutionData;
 use core::fmt::Debug;
@@ -164,3 +167,28 @@ impl ExecutionPayload for op_alloy_rpc_types_engine::OpExecutionData {
         self.payload.as_v1().gas_used
     }
 }
+
+/// Special implementation for Ethereum types that provides additional helper methods
+impl<'a, Attributes> PayloadOrAttributes<'a, ExecutionData, Attributes>
+where
+    Attributes: PayloadAttributes,
+{
+    /// Return the execution requests from the payload, if available.
+    ///
+    /// This will return `Some(requests)` only if:
+    /// - The payload is an `ExecutionData` (not `PayloadAttributes`)
+    /// - The payload has Prague payload fields
+    /// - The Prague fields contain requests (not a hash)
+    ///
+    /// Returns `None` in all other cases.
+    pub fn execution_requests(&self) -> Option<&Requests> {
+        if let Self::ExecutionPayload(payload) = self {
+            payload.sidecar.prague().and_then(|prague_fields| match &prague_fields.requests {
+                RequestsOrHash::Requests(requests) => Some(requests),
+                _ => None,
+            })
+        } else {
+            None
+        }
+    }
+}