fix(l1): return invalid payload for invalid transactions in new payload endpoint (#2561)

**Motivation**
This fixes the failing consume-engine hive tests on Prague and Cancun
<!-- Why does this pull request exist? What are its goals? -->

**Description**
* Return `INVALID` payload status when we fail to decode transactions as
per spec ([Step 6 item
1](https://github.com/ethereum/execution-apis/blob/main/src/engine/paris.md#engine_newpayloadv1))
<!-- A clear and concise general description of the changes this PR
introduces -->

<!-- Link to issues: Resolves #111, Resolves #222 -->

Closes #issue_number

Author: fmoletta

crates/networking/rpc/engine/payload.rs

@@ -5,6 +5,7 @@ use ethrex_common::types::requests::{compute_requests_hash, EncodedRequests};
 use ethrex_common::types::{Block, BlockBody, BlockHash, BlockNumber, Fork};
 use ethrex_common::{H256, U256};
 use ethrex_p2p::sync::SyncMode;
+use ethrex_rlp::error::RLPDecodeError;
 use serde_json::Value;
 use tracing::{error, info, warn};
 
@@ -34,7 +35,14 @@ impl RpcHandler for NewPayloadV1Request {
 
     async fn handle(&self, context: RpcApiContext) -> Result<Value, RpcErr> {
         validate_execution_payload_v1(&self.payload)?;
-        let block = get_block_from_payload(&self.payload, None, None)?;
+        let block = match get_block_from_payload(&self.payload, None, None) {
+            Ok(block) => block,
+            Err(err) => {
+                return Ok(serde_json::to_value(PayloadStatus::invalid_with_err(
+                    &err.to_string(),
+                ))?)
+            }
+        };
         let payload_status = handle_new_payload_v1_v2(&self.payload, block, context).await?;
         serde_json::to_value(payload_status).map_err(|error| RpcErr::Internal(error.to_string()))
     }
@@ -59,8 +67,14 @@ impl RpcHandler for NewPayloadV2Request {
             // Behave as a v1
             validate_execution_payload_v1(&self.payload)?;
         }
-
-        let block = get_block_from_payload(&self.payload, None, None)?;
+        let block = match get_block_from_payload(&self.payload, None, None) {
+            Ok(block) => block,
+            Err(err) => {
+                return Ok(serde_json::to_value(PayloadStatus::invalid_with_err(
+                    &err.to_string(),
+                ))?)
+            }
+        };
         let payload_status = handle_new_payload_v1_v2(&self.payload, block, context).await?;
         serde_json::to_value(payload_status).map_err(|error| RpcErr::Internal(error.to_string()))
     }
@@ -105,8 +119,18 @@ impl RpcHandler for NewPayloadV3Request {
     }
 
     async fn handle(&self, context: RpcApiContext) -> Result<Value, RpcErr> {
-        let block =
-            get_block_from_payload(&self.payload, Some(self.parent_beacon_block_root), None)?;
+        let block = match get_block_from_payload(
+            &self.payload,
+            Some(self.parent_beacon_block_root),
+            None,
+        ) {
+            Ok(block) => block,
+            Err(err) => {
+                return Ok(serde_json::to_value(PayloadStatus::invalid_with_err(
+                    &err.to_string(),
+                ))?)
+            }
+        };
         validate_fork(&block, Fork::Cancun, &context)?;
         validate_execution_payload_v3(&self.payload)?;
         let payload_status = handle_new_payload_v3(
@@ -205,11 +229,18 @@ impl RpcHandler for NewPayloadV4Request {
         validate_execution_requests(&self.execution_requests)?;
 
         let requests_hash = compute_requests_hash(&self.execution_requests);
-        let block = get_block_from_payload(
+        let block = match get_block_from_payload(
             &self.payload,
             Some(self.parent_beacon_block_root),
             Some(requests_hash),
-        )?;
+        ) {
+            Ok(block) => block,
+            Err(err) => {
+                return Ok(serde_json::to_value(PayloadStatus::invalid_with_err(
+                    &err.to_string(),
+                ))?)
+            }
+        };
 
         let chain_config = context.storage.get_chain_config()?;
 
@@ -645,14 +676,13 @@ fn get_block_from_payload(
     payload: &ExecutionPayload,
     parent_beacon_block_root: Option<H256>,
     requests_hash: Option<H256>,
-) -> Result<Block, RpcErr> {
+) -> Result<Block, RLPDecodeError> {
     let block_hash = payload.block_hash;
     info!("Received new payload with block hash: {block_hash:#x}");
 
     payload
         .clone()
         .into_block(parent_beacon_block_root, requests_hash)
-        .map_err(|error| RpcErr::Internal(error.to_string()))
 }
 
 fn validate_block_hash(payload: &ExecutionPayload, block: &Block) -> Result<(), RpcErr> {