Remove slog-scope from common dependencies & simplify slog features

Only client crates should specifies the log level features and a library
should not use `slog-scope` but instead allow to pass the logger to let
the client specify what it wants (ie: he may want a logger specially for
a given task and use a global logger otherwise).

Author: Alenar

mithril-aggregator/src/dependency_injection/builder.rs

@@ -534,6 +534,7 @@ impl DependenciesBuilder {
             &self.configuration.data_stores_directory,
             &format!("immutables_digests_{}.json", self.configuration.network),
         )
+        .with_logger(self.get_logger().await?)
         .should_reset_digests_cache(self.configuration.reset_digests_cache)
         .build()
         .await?;

mithril-common/Cargo.toml

@@ -44,11 +44,7 @@ serde_json = "1.0.107"
 serde_with = "3.3.0"
 serde_yaml = "0.9.25"
 sha2 = "0.10.8"
-slog = { version = "2.7.0", features = [
-    "max_level_trace",
-    "release_max_level_debug",
-] }
-slog-scope = "4.4.0"
+slog = "2.7.0"
 sqlite = { version = "0.31.1", features = ["bundled"] }
 strum = { version = "0.25.0", features = ["derive"] }
 thiserror = "1.0.49"

mithril-common/src/digesters/cache/json_provider_builder.rs

@@ -3,7 +3,7 @@ use crate::{
     StdResult,
 };
 use anyhow::Context;
-use slog_scope::info;
+use slog::{info, Logger};
 use std::path::Path;
 use tokio::fs;
 
@@ -13,6 +13,7 @@ pub struct JsonImmutableFileDigestCacheProviderBuilder<'a> {
     filename: &'a str,
     ensure_dir_exist: bool,
     reset_digests_cache: bool,
+    logger: Logger,
 }
 
 impl<'a> JsonImmutableFileDigestCacheProviderBuilder<'a> {
@@ -23,6 +24,7 @@ impl<'a> JsonImmutableFileDigestCacheProviderBuilder<'a> {
             filename,
             ensure_dir_exist: false,
             reset_digests_cache: false,
+            logger: Logger::root(slog::Discard, slog::o!()),
         }
     }
 
@@ -38,6 +40,12 @@ impl<'a> JsonImmutableFileDigestCacheProviderBuilder<'a> {
         self
     }
 
+    /// Set the [Logger] to use.
+    pub fn with_logger(&mut self, logger: Logger) -> &mut Self {
+        self.logger = logger;
+        self
+    }
+
     /// Build a [JsonImmutableFileDigestCacheProvider] based on the parameters previously set.
     pub async fn build(&self) -> StdResult<JsonImmutableFileDigestCacheProvider> {
         let cache_file = self.cache_dir.join(self.filename);
@@ -62,6 +70,7 @@ impl<'a> JsonImmutableFileDigestCacheProviderBuilder<'a> {
         }
 
         info!(
+            self.logger,
             "Storing/Getting immutables digests cache from: {}",
             cache_file.display()
         );

mithril-signer/src/runtime/signer_services.rs

@@ -130,6 +130,7 @@ impl<'a> ProductionServiceBuilder<'a> {
             &format!("immutables_digests_{}.json", self.config.network),
         )
         .should_reset_digests_cache(self.config.reset_digests_cache)
+        .with_logger(slog_scope::logger())
         .build()
         .await?;