fix(l1): fixed proper parent when receiveing a NewPayloadRequest (#2690)

**Motivation**

This pr aims to fix a bug with the parent assigned in NewPayloadRequest
if the parent is valid. From the [Paris fork
documentation](https://github.com/ethereum/execution-apis/blob/main/src/engine/paris.md)
"latestValidHash: DATA|null, 32 Bytes - the hash of the most recent
valid block in the branch defined by payload and its ancestors"

**Description**

Removed storage get for the canonical latest valid ancestor and replaced
with the parent hash (if it's valid).

Fixes 27 tests in #1285 in "engine-cancun"

Author: fedacking

.github/workflows/pr-main_l1.yaml

@@ -155,7 +155,7 @@ jobs:
             ethrex_flags: ""
           - name: "Cancun Engine tests"
             simulation: ethereum/engine
-            test_pattern: "engine-cancun/Blob Transactions On Block 1|Blob Transaction Ordering, Single|Blob Transaction Ordering, Multiple Accounts|Replace Blob Transactions|Parallel Blob Transactions|ForkchoiceUpdatedV3|ForkchoiceUpdatedV2|ForkchoiceUpdated Version|GetPayload|NewPayloadV3 After Cancun|NewPayloadV3 Before Cancun|NewPayloadV3 Versioned Hashes|Incorrect BlobGasUsed|ParentHash equals BlockHash|RPC:|in ForkchoiceState|Unknown SafeBlockHash|Unknown FinalizedBlockHash|Unique|Re-Execute Payload|Multiple New Payloads|NewPayload with|Build Payload with|Re-org to Previously|Safe Re-Org to Side Chain|Transaction Re-Org|Re-Org Back into Canonical Chain, Depth=5|Suggested Fee Recipient Test|PrevRandao Opcode|Fork ID: *|Request Blob Pooled Transactions|Invalid NewPayload, Incomplete Transactions|Re-Org Back to Canonical Chain*|Invalid PayloadAttributes*|Invalid NewPayload, VersionedHashes|Invalid NewPayload, Incomplete VersionedHashes|Invalid NewPayload, Extra VersionedHashes|Bad Hash on NewPayload|Unknown HeadBlockHash|In-Order Consecutive Payload Execution|Valid NewPayload->ForkchoiceUpdated|Invalid NewPayload, ParentHash|Syncing=False|Payload Build after New Invalid Payload" # Invalid Missing Ancestor Syncing ReOrg|Invalid P9|Invalid P10 -> flaky
+            test_pattern: "engine-cancun/Blob Transactions On Block 1|Blob Transaction Ordering, Single|Blob Transaction Ordering, Multiple Accounts|Replace Blob Transactions|Parallel Blob Transactions|ForkchoiceUpdatedV3|ForkchoiceUpdatedV2|ForkchoiceUpdated Version|GetPayload|NewPayloadV3 After Cancun|NewPayloadV3 Before Cancun|NewPayloadV3 Versioned Hashes|Incorrect BlobGasUsed|ParentHash equals BlockHash|RPC:|in ForkchoiceState|Unknown SafeBlockHash|Unknown FinalizedBlockHash|Unique|Re-Execute Payload|Multiple New Payloads|NewPayload with|Build Payload with|Re-org to Previously|Safe Re-Org to Side Chain|Transaction Re-Org|Re-Org Back into Canonical Chain, Depth=5|Suggested Fee Recipient Test|PrevRandao Opcode|Fork ID: *|Request Blob Pooled Transactions|Invalid NewPayload, Incomplete Transactions|Re-Org Back to Canonical Chain*|Invalid PayloadAttributes*|Invalid NewPayload, VersionedHashes|Invalid NewPayload, Incomplete VersionedHashes|Invalid NewPayload, Extra VersionedHashes|Bad Hash on NewPayload|Unknown HeadBlockHash|In-Order Consecutive Payload Execution|Valid NewPayload->ForkchoiceUpdated|Invalid NewPayload, ParentHash|Syncing=False|Payload Build after New Invalid Payload|Invalid NewPayload|Invalid Missing Ancestor"
             ethrex_flags: ""
           - name: "Paris Engine tests"
             simulation: ethereum/engine

crates/networking/rpc/engine/payload.rs

@@ -622,13 +622,16 @@ async fn handle_new_payload_v1_v2(
         return Ok(status);
     }
 
+    // We have validated ancestors, the parent is correct
+    let latest_valid_hash = block.header.parent_hash;
+
     if context.syncer.sync_mode() == SyncMode::Snap {
         warn!("Snap sync in progress, skipping new payload validation");
         return Ok(PayloadStatus::syncing());
     }
 
     // All checks passed, execute payload
-    let payload_status = try_execute_payload(&block, &context).await?;
+    let payload_status = try_execute_payload(&block, &context, latest_valid_hash).await?;
     Ok(payload_status)
 }
 
@@ -699,6 +702,7 @@ fn validate_block_hash(payload: &ExecutionPayload, block: &Block) -> Result<(),
 async fn try_execute_payload(
     block: &Block,
     context: &RpcApiContext,
+    latest_valid_hash: H256,
 ) -> Result<PayloadStatus, RpcErr> {
     let block_hash = block.hash();
     let storage = &context.storage;
@@ -710,16 +714,6 @@ async fn try_execute_payload(
     // Execute and store the block
     info!("Executing payload with block hash: {block_hash:#x}");
 
-    // TODO: this is not correct, the block being validated it no necesarily a descendant
-    // of the latest canonical block
-    let latest_valid_hash = context
-        .storage
-        .get_latest_canonical_block_hash()
-        .await?
-        .ok_or(RpcErr::Internal(
-            "Missing latest canonical block".to_owned(),
-        ))?;
-
     match context.blockchain.add_block(block).await {
         Err(ChainError::ParentNotFound) => {
             // Start sync