Some more TODOs

beacon_node/beacon_chain/src/payload_envelope_verification/execution_pending_envelope.rs

@@ -44,6 +44,8 @@ impl<T: BeaconChainTypes> IntoExecutionPendingEnvelope<T> for GossipVerifiedEnve
         let envelope = &signed_envelope.message;
         let payload = &envelope.payload;
 
+        // TODO(gloas)
+
         // Verify the execution payload is valid
         let payload_notifier = PayloadNotifier::new(
             chain.clone(),

beacon_node/beacon_chain/src/payload_envelope_verification/payload_notifier.rs

@@ -29,6 +29,7 @@ impl<T: BeaconChainTypes> PayloadNotifier<T> {
         let payload_verification_status = {
             let payload_message = &envelope.message;
 
+            // TODO(gloas) re-asses if optimistic syncing works similarly post-gloas
             match notify_execution_layer {
                 NotifyExecutionLayer::No if chain.config.optimistic_finalized_sync => {
                     let new_payload_request = Self::build_new_payload_request(&envelope, &block)?;

beacon_node/network/src/network_beacon_processor/gossip_methods.rs

@@ -3272,7 +3272,8 @@ impl<T: BeaconChainTypes> NetworkBeaconProcessor<T> {
             Span::current().record("beacon_block_root", beacon_block_root.to_string());
 
             // TODO(gloas) in process_gossip_block here we check_and_insert on the duplicate cache
-            // before calling gossip_verified_block
+            // before calling gossip_verified_block. We need this to ensure we dont try to execute the
+            // payload multiple times.
 
             self.process_gossip_verified_execution_payload_envelope(
                 peer_id,