From b0b3b5b8c1a160c4eea3768924702a3db6627470 Mon Sep 17 00:00:00 2001
From: Ethan Brierley <ethanboxx@gmail.com>
Date: Wed, 28 Aug 2024 22:37:11 +0100
Subject: [PATCH] Revert "Simplify"

This reverts commit d5290d056f2303cfbceaf539b9cb41f97b1bd6f4.
---
 reqwest-retry/Cargo.toml        |  2 ++
 reqwest-retry/src/middleware.rs | 41 ++++++++++++++++++++++++---------
 2 files changed, 32 insertions(+), 11 deletions(-)

diff --git a/reqwest-retry/Cargo.toml b/reqwest-retry/Cargo.toml
index 7fc6e5c..41f51fd 100644
--- a/reqwest-retry/Cargo.toml
+++ b/reqwest-retry/Cargo.toml
@@ -11,6 +11,7 @@ categories = ["web-programming::http-client"]
 
 [features]
 default = ["tracing"]
+log = ["dep:log"]
 tracing = ["dep:tracing"]
 
 [dependencies]
@@ -19,6 +20,7 @@ reqwest-middleware = { version = "0.3.0", path = "../reqwest-middleware" }
 anyhow = "1.0.0"
 async-trait = "0.1.51"
 futures = "0.3.0"
+log = { version = "0.4.22", optional = true }
 http = "1.0"
 reqwest = { version = "0.12.0", default-features = false }
 retry-policies = "0.4"
diff --git a/reqwest-retry/src/middleware.rs b/reqwest-retry/src/middleware.rs
index 66daec6..a63764e 100644
--- a/reqwest-retry/src/middleware.rs
+++ b/reqwest-retry/src/middleware.rs
@@ -9,6 +9,16 @@ use reqwest::{Request, Response};
 use reqwest_middleware::{Error, Middleware, Next, Result};
 use retry_policies::RetryPolicy;
 
+#[cfg(feature = "tracing")]
+type LogLevel = ::tracing::Level;
+#[cfg(feature = "tracing")]
+use ::tracing as LogTracing;
+
+#[cfg(all(feature = "log", not(feature = "tracing")))]
+type LogLevel = ::log::Level;
+#[cfg(all(feature = "log", not(feature = "tracing")))]
+use ::log as LogTracing;
+
 #[doc(hidden)]
 // We need this macro because tracing expects the level to be const:
 // https://github.com/tokio-rs/tracing/issues/2730
@@ -16,15 +26,22 @@ use retry_policies::RetryPolicy;
 macro_rules! log_retry {
     ($level:expr, $($args:tt)*) => {{
         match $level {
-            ::tracing::Level::TRACE => ::tracing::trace!($($args)*),
-            ::tracing::Level::DEBUG => ::tracing::debug!($($args)*),
-            ::tracing::Level::INFO => ::tracing::info!($($args)*),
-            ::tracing::Level::WARN => ::tracing::warn!($($args)*),
-            ::tracing::Level::ERROR => ::tracing::error!($($args)*),
+            LogLevel::TRACE => LogTracing::trace!($($args)*),
+            LogLevel::DEBUG => LogTracing::debug!($($args)*),
+            LogLevel::INFO => LogTracing::info!($($args)*),
+            LogLevel::WARN => LogTracing::warn!($($args)*),
+            LogLevel::ERROR => LogTracing::error!($($args)*),
         }
     }};
 }
 
+#[cfg(all(feature = "log", not(feature = "tracing")))]
+macro_rules! log_retry {
+    ($level:expr, $($args:tt)*) => {{
+        LogTracing::log!($level, $($args)*)
+    }};
+}
+
 /// `RetryTransientMiddleware` offers retry logic for requests that fail in a transient manner
 /// and can be safely executed again.
 ///
@@ -70,8 +87,8 @@ pub struct RetryTransientMiddleware<
 > {
     retry_policy: T,
     retryable_strategy: R,
-    #[cfg(feature = "tracing")]
-    retry_log_level: ::tracing::Level,
+    #[cfg(any(feature = "log", feature = "tracing"))]
+    retry_log_level: LogLevel,
 }
 
 impl<T: RetryPolicy + Send + Sync> RetryTransientMiddleware<T, DefaultRetryableStrategy> {
@@ -82,8 +99,8 @@ impl<T: RetryPolicy + Send + Sync> RetryTransientMiddleware<T, DefaultRetryableS
 
     /// Set the log [level][tracing::Level] for retry events.
     /// The default is [`WARN`][tracing::Level::WARN].
-    #[cfg(feature = "tracing")]
-    pub fn with_retry_log_level(mut self, level: ::tracing::Level) -> Self {
+    #[cfg(any(feature = "log", feature = "tracing"))]
+    pub fn with_retry_log_level(mut self, level: LogLevel) -> Self {
         self.retry_log_level = level;
         self
     }
@@ -100,7 +117,9 @@ where
             retry_policy,
             retryable_strategy,
             #[cfg(feature = "tracing")]
-            retry_log_level: ::tracing::Level::WARN,
+            retry_log_level: LogLevel::WARN,
+            #[cfg(all(feature = "log", not(feature = "tracing")))]
+            retry_log_level: LogLevel::Warn,
         }
     }
 }
@@ -167,7 +186,7 @@ where
                             .duration_since(SystemTime::now())
                             .unwrap_or_else(|_| Duration::default());
                         // Sleep the requested amount before we try again.
-                        #[cfg(feature = "tracing")]
+                        #[cfg(any(feature = "log", feature = "tracing"))]
                         log_retry!(
                             self.retry_log_level,
                             "Retry attempt #{}. Sleeping {:?} before the next attempt",