diff --git a/crates/ev-precompiles/src/mint.rs b/crates/ev-precompiles/src/mint.rs index 6a499c8..fd33cdf 100644 --- a/crates/ev-precompiles/src/mint.rs +++ b/crates/ev-precompiles/src/mint.rs @@ -144,16 +144,16 @@ impl MintPrecompile { caller: Address, ) -> MintPrecompileResult<()> { if caller == self.admin { - tracing::debug!(target: "mint_precompile", ?caller, "authorization granted: admin"); + tracing::debug!(target: "mint_precompile", %caller, "authorization granted: admin"); return Ok(()); } let allowlisted = Self::is_allowlisted(internals, caller)?; if allowlisted { - tracing::debug!(target: "mint_precompile", ?caller, "authorization granted: allowlist"); + tracing::debug!(target: "mint_precompile", %caller, "authorization granted: allowlist"); Ok(()) } else { - tracing::warn!(target: "mint_precompile", ?caller, "authorization denied: not admin and not allowlisted"); + tracing::warn!(target: "mint_precompile", %caller, "authorization denied: not admin and not allowlisted"); Err(MintPrecompileError::halt_static("unauthorized caller")) } } @@ -171,7 +171,7 @@ impl MintPrecompile { let allowlisted = !raw_value.is_zero(); tracing::debug!( target: "mint_precompile", - ?addr, + %addr, slot = %key, value = %raw_value, allowlisted, @@ -215,7 +215,7 @@ impl Precompile for MintPrecompile { tracing::info!( target: "mint_precompile", - ?caller, + %caller, gas = gas_limit, calldata_len = data_len, "mint precompile call invoked" diff --git a/crates/node/src/builder.rs b/crates/node/src/builder.rs index 27398d6..2a57e6c 100644 --- a/crates/node/src/builder.rs +++ b/crates/node/src/builder.rs @@ -52,7 +52,7 @@ where if let Some((sink, activation)) = config.base_fee_redirect_settings() { info!( target: "ev-reth", - fee_sink = ?sink, + fee_sink = %sink, activation_height = activation, "Base fee redirect enabled via chainspec" ); @@ -69,7 +69,7 @@ where #[instrument(skip(self, attributes), fields( parent_hash = %attributes.parent_hash, tx_count = attributes.transactions.len(), - gas_limit = ?attributes.gas_limit, + gas_limit = attributes.gas_limit.unwrap_or(0), duration_ms = tracing::field::Empty, ))] pub async fn build_payload( @@ -118,7 +118,7 @@ where suggested_fee_recipient = sink; info!( target: "ev-reth", - fee_sink = ?sink, + fee_sink = %sink, block_number, "Suggested fee recipient missing; defaulting to base-fee sink" ); @@ -173,7 +173,7 @@ where debug!(gas_used = ?gas_used, "transaction executed successfully"); } Err(err) => { - tracing::warn!(error = ?err, tx_hash = %tx.tx_hash(), "transaction execution failed"); + tracing::warn!(error = %err, tx_hash = %tx.tx_hash(), "transaction execution failed"); } } } @@ -192,7 +192,7 @@ where info!( block_number = sealed_block.number, - block_hash = ?sealed_block.hash(), + block_hash = %sealed_block.hash(), tx_count = sealed_block.transaction_count(), gas_used = sealed_block.gas_used, "built block" @@ -221,13 +221,13 @@ where let config = match EvolvePayloadBuilderConfig::from_chain_spec(&chain_spec) { Ok(config) => config, Err(err) => { - tracing::warn!(target: "ev-reth", error = ?err, "Failed to parse chainspec extras"); + tracing::warn!(target: "ev-reth", error = %err, "Failed to parse chainspec extras"); return None; } }; if let Err(err) = config.validate() { - tracing::warn!(target: "ev-reth", error = ?err, "Invalid evolve payload builder configuration"); + tracing::warn!(target: "ev-reth", error = %err, "Invalid evolve payload builder configuration"); return None; } diff --git a/crates/node/src/executor.rs b/crates/node/src/executor.rs index 70dbd18..082a736 100644 --- a/crates/node/src/executor.rs +++ b/crates/node/src/executor.rs @@ -411,7 +411,7 @@ where .map(|(sink, activation)| { info!( target = "ev-reth::executor", - fee_sink = ?sink, + fee_sink = %sink, activation_height = activation, "Base fee redirect enabled" ); diff --git a/crates/node/src/payload_service.rs b/crates/node/src/payload_service.rs index e58122e..deb4832 100644 --- a/crates/node/src/payload_service.rs +++ b/crates/node/src/payload_service.rs @@ -45,7 +45,7 @@ impl EvolvePayloadBuilderBuilder { /// Create a new builder with evolve args. pub fn new() -> Self { let config = EvolvePayloadBuilderConfig::new(); - info!("Created Evolve payload builder with config: {:?}", config); + info!("created evolve payload builder"); Self { config } } } @@ -138,7 +138,7 @@ where if let Some(sink) = self.config.base_fee_sink_for_block(block_number) { info!( target: "ev-reth", - fee_sink = ?sink, + fee_sink = %sink, block_number, "Suggested fee recipient missing; defaulting to base-fee sink" ); diff --git a/crates/node/src/validator.rs b/crates/node/src/validator.rs index 1bc3f81..4d3165d 100644 --- a/crates/node/src/validator.rs +++ b/crates/node/src/validator.rs @@ -80,7 +80,7 @@ impl PayloadValidator for EvolveEngineValidator { .map_err(|e| NewPayloadError::Other(e.into())) } Err(err) => { - debug!(error = ?err, "payload validation error"); + debug!(error = %err, "payload validation error"); // Check if this is an error we can bypass for evolve: // 1. BlockHash mismatch - ev-reth computes different hash due to custom tx types @@ -96,7 +96,7 @@ impl PayloadValidator for EvolveEngineValidator { || is_unknown_tx_type_error(&err); if should_bypass { - info!(error = ?err, "bypassing validation error for ev-reth"); + info!(error = %err, "bypassing validation error for ev-reth"); // For evolve, we trust the payload builder - parse the block with EvNode support. let ev_block = parse_evolve_payload(payload)?; Span::current().record("block_hash", tracing::field::display(ev_block.hash()));