chore: fixes

hyperlane-xyz · Dec 29, 2024 · a8c9662 · a8c9662
1 parent 1d4177f
commit a8c9662
Show file tree

Hide file tree

Showing 6 changed files with 85 additions and 40 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/ethers-middleware/Cargo.toml b/ethers-middleware/Cargo.toml
@@ -29,6 +29,7 @@ futures-util = { version = "^0.3" }
 futures-locks = { version = "0.7", default-features = false }
 tracing = { version = "0.1.37", default-features = false }
 tracing-futures = { version = "0.2.5", default-features = false }
+tracing-test = "0.2.2"
 
 # for gas oracles
 reqwest = { version = "0.11.13", default-features = false, features = ["json", "rustls-tls"] }

diff --git a/ethers-middleware/src/gas_oracle/middleware.rs b/ethers-middleware/src/gas_oracle/middleware.rs
@@ -3,6 +3,7 @@ use async_trait::async_trait;
 use ethers_core::types::{transaction::eip2718::TypedTransaction, *};
 use ethers_providers::{FromErr, Middleware, PendingTransaction};
 use thiserror::Error;
+use tracing::instrument;
 
 #[derive(Debug)]
 /// Middleware used for fetching gas prices over an API instead of `eth_gasPrice`
@@ -102,13 +103,14 @@ where
         Ok(self.gas_oracle.estimate_eip1559_fees().await?)
     }
 
-    #[instrument(skip(self), name = "GasOracle::send_transaction")]
+    #[instrument(skip(self, tx), name = "GasOracle::send_transaction")]
     async fn send_transaction<T: Into<TypedTransaction> + Send + Sync>(
         &self,
         tx: T,
         block: Option<BlockId>,
     ) -> Result<PendingTransaction<'_, Self::Provider>, Self::Error> {
         let mut tx = tx.into();
+        tracing::debug!(?tx, "Sending transaction");
         self.fill_transaction(&mut tx, block).await?;
         self.inner.send_transaction(tx, block).await.map_err(MiddlewareError::MiddlewareError)
     }