ref: remove some default traces + cfg feature on tracing for instrument

Author: LeoPatOZ

src/robust_provider/provider.rs

@@ -12,6 +12,7 @@ use backon::{ExponentialBuilder, Retryable};
 use futures::TryFutureExt;
 use thiserror::Error;
 use tokio::time::{error as TokioError, timeout};
+#[cfg(feature = "tracing")]
 use tracing::instrument;
 
 use crate::robust_provider::RobustSubscription;
@@ -322,7 +323,7 @@ impl<N: Network> RobustProvider<N> {
             .await
     }
 
-    #[instrument(level = "trace", skip(self, operation, last_error))]
+    #[cfg_attr(feature = "tracing", instrument(level = "trace", skip(self, operation, last_error)))]
     pub(crate) async fn try_fallback_providers<T: Debug, F, Fut>(
         &self,
         operation: F,
@@ -338,7 +339,7 @@ impl<N: Network> RobustProvider<N> {
             .map(|(value, _idx)| value)
     }
 
-    #[instrument(level = "trace", skip(self, operation, last_error))]
+    #[cfg_attr(feature = "tracing", instrument(level = "trace", skip(self, operation, last_error)))]
     pub(crate) async fn try_fallback_providers_from<T: Debug, F, Fut>(
         &self,
         operation: F,
@@ -362,7 +363,7 @@ impl<N: Network> RobustProvider<N> {
                 continue;
             }
 
-            tracing::trace!(
+            trace!(
                 fallback_provider_index = fallback_idx + 1,
                 total_num_fallbacks = num_fallbacks,
                 "Attempting fallback provider"
@@ -384,7 +385,7 @@ impl<N: Network> RobustProvider<N> {
             }
         }
 
-        tracing::error!("All providers exhausted");
+        tracing::error!("All providers failed");
 
         Err(last_error)
     }

src/robust_provider/subscription.rs

@@ -4,6 +4,7 @@ use std::{
     task::{Context, Poll, ready},
     time::{Duration, Instant},
 };
+#[cfg(feature = "tracing")]
 use tracing::instrument;
 
 use alloy::{
@@ -133,7 +134,7 @@ impl<N: Network> RobustSubscription<N> {
 
     /// Try to reconnect to the primary provider if enough time has elapsed.
     /// Returns true if reconnection was successful, false if it's not time yet or if it failed.
-    #[instrument(level = "trace", skip(self))]
+    #[cfg_attr(feature = "tracing", instrument(level = "trace", skip(self)))]
     async fn try_reconnect_to_primary(&mut self, force: bool) -> bool {
         // Check if we should attempt reconnection
         let should_reconnect = force ||
@@ -148,8 +149,6 @@ impl<N: Network> RobustSubscription<N> {
             return false;
         }
 
-        trace!("Attempting to reconnect to primary provider");
-
         let operation =
             move |provider: RootProvider<N>| async move { provider.subscribe_blocks().await };