From 6e89ba63be49ff34e4ab70a9ba82b2aee4841f98 Mon Sep 17 00:00:00 2001
From: Eitan Seri- Levi <eserilev@gmail.com>
Date: Tue, 24 Feb 2026 10:51:09 -0800
Subject: [PATCH] Added slot to logs

---
 beacon_node/beacon_chain/src/execution_payload.rs    | 12 ++++--------
 .../payload_notifier.rs                              |  2 +-
 2 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/beacon_node/beacon_chain/src/execution_payload.rs b/beacon_node/beacon_chain/src/execution_payload.rs
index b2d00a530a..cf6c5d83b4 100644
--- a/beacon_node/beacon_chain/src/execution_payload.rs
+++ b/beacon_node/beacon_chain/src/execution_payload.rs
@@ -118,6 +118,7 @@ impl<T: BeaconChainTypes> PayloadNotifier<T> {
             notify_new_payload(
                 &self.chain,
                 self.block.message().tree_hash_root(),
+                self.block.message().slot(),
                 self.block.message().try_into()?,
             )
             .await
@@ -137,6 +138,7 @@ impl<T: BeaconChainTypes> PayloadNotifier<T> {
 pub async fn notify_new_payload<T: BeaconChainTypes>(
     chain: &Arc<BeaconChain<T>>,
     beacon_block_root: Hash256,
+    slot: Slot,
     new_payload_request: NewPayloadRequest<'_, T::EthSpec>,
 ) -> Result<PayloadVerificationStatus, BlockError> {
     let execution_layer = chain
@@ -163,11 +165,8 @@ pub async fn notify_new_payload<T: BeaconChainTypes>(
                     ?validation_error,
                     ?latest_valid_hash,
                     ?execution_block_hash,
-                    // TODO(gloas) are these other logs important?
                     root = ?beacon_block_root,
-                    // graffiti = block.body().graffiti().as_utf8_lossy(),
-                    // proposer_index = block.proposer_index(),
-                    // slot = %block.slot(),
+                    %slot,
                     method = "new_payload",
                     "Invalid execution payload"
                 );
@@ -209,11 +208,8 @@ pub async fn notify_new_payload<T: BeaconChainTypes>(
                 warn!(
                     ?validation_error,
                     ?execution_block_hash,
-                    // TODO(gloas) are these other logs important?
                     root = ?beacon_block_root,
-                    // graffiti = block.body().graffiti().as_utf8_lossy(),
-                    // proposer_index = block.proposer_index(),
-                    // slot = %block.slot(),
+                    %slot,
                     method = "new_payload",
                     "Invalid execution payload block hash"
                 );
diff --git a/beacon_node/beacon_chain/src/payload_envelope_verification/payload_notifier.rs b/beacon_node/beacon_chain/src/payload_envelope_verification/payload_notifier.rs
index 5b1f332b5a..a468bc5bc4 100644
--- a/beacon_node/beacon_chain/src/payload_envelope_verification/payload_notifier.rs
+++ b/beacon_node/beacon_chain/src/payload_envelope_verification/payload_notifier.rs
@@ -62,7 +62,7 @@ impl<T: BeaconChainTypes> PayloadNotifier<T> {
         } else {
             let block_root = self.envelope.message.beacon_block_root;
             let request = Self::build_new_payload_request(&self.envelope, &self.block)?;
-            notify_new_payload(&self.chain, block_root, request).await
+            notify_new_payload(&self.chain, block_root, self.envelope.slot(), request).await
         }
     }