From 22e18ac2d9935511e1de20c361d5c385ffef1a6c Mon Sep 17 00:00:00 2001 From: Matthias Seitz Date: Mon, 10 Jun 2024 22:02:41 +0200 Subject: [PATCH] fix: dont panic on invalud fcu response (#8725) --- crates/consensus/debug-client/src/client.rs | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/crates/consensus/debug-client/src/client.rs b/crates/consensus/debug-client/src/client.rs index d627b6301..a63b7a85d 100644 --- a/crates/consensus/debug-client/src/client.rs +++ b/crates/consensus/debug-client/src/client.rs @@ -130,17 +130,20 @@ impl DebugConsensusClient

{ continue; } }; - reth_rpc_api::EngineApiClient::::fork_choice_updated_v3( + let state = reth_rpc_types::engine::ForkchoiceState { + head_block_hash: block_hash, + safe_block_hash, + finalized_block_hash, + }; + let _ = reth_rpc_api::EngineApiClient::::fork_choice_updated_v3( &execution_client, - reth_rpc_types::engine::ForkchoiceState { - head_block_hash: block_hash, - safe_block_hash, - finalized_block_hash, - }, + state, None, ) .await - .unwrap(); + .inspect_err(|err| { + warn!(target: "consensus::debug-client", %err, ?state, "failed to submit fork choice update to execution client"); + }); } } }