feat: [#591] move log_level into a new logging section in config

Old:

```toml
log_level = "info"

[website]
name = "Torrust"
```

New:

```toml
[logging]
log_level = "info"

[website]
name = "Torrust"
```

And the value is not Optional anymore. It was a Optional<LogLevel> but
when it was None it defaulted to LogLevel::Info. In practice that means
is mandatory but with the `Info` default value.

Author: josecelano

share/default/config/index.container.mysql.toml

@@ -1,3 +1,4 @@
+[logging]
 log_level = "info"
 
 [database]

share/default/config/index.container.sqlite3.toml

@@ -1,3 +1,4 @@
+[logging]
 log_level = "info"
 
 [database]

share/default/config/index.development.sqlite3.toml

@@ -1,3 +1,4 @@
+[logging]
 log_level = "info"
 
 # Uncomment if you want to enable TSL for development

share/default/config/index.private.e2e.container.sqlite3.toml

@@ -1,3 +1,4 @@
+[logging]
 log_level = "info"
 
 [tracker]

share/default/config/index.public.e2e.container.mysql.toml

@@ -1,3 +1,4 @@
+[logging]
 log_level = "info"
 
 [tracker]

share/default/config/index.public.e2e.container.sqlite3.toml

@@ -1,3 +1,4 @@
+[logging]
 log_level = "info"
 
 [tracker]

src/app.rs

@@ -38,7 +38,7 @@ pub struct Running {
 /// It panics if there is an error connecting to the database.
 #[allow(clippy::too_many_lines)]
 pub async fn run(configuration: Configuration, api_version: &Version) -> Running {
-    let log_level = configuration.settings.read().await.log_level.clone();
+    let log_level = configuration.settings.read().await.logging.log_level.clone();
 
     logging::setup(&log_level);
 

src/bootstrap/logging.rs

@@ -11,12 +11,12 @@ use std::sync::Once;
 use tracing::info;
 use tracing::level_filters::LevelFilter;
 
-use crate::config::v1::LogLevel;
+use crate::config::v1::logging::LogLevel;
 
 static INIT: Once = Once::new();
 
-pub fn setup(log_level: &Option<LogLevel>) {
-    let tracing_level = config_level_or_default(log_level);
+pub fn setup(log_level: &LogLevel) {
+    let tracing_level: LevelFilter = log_level.clone().into();
 
     if tracing_level == LevelFilter::OFF {
         return;
@@ -27,20 +27,6 @@ pub fn setup(log_level: &Option<LogLevel>) {
     });
 }
 
-fn config_level_or_default(log_level: &Option<LogLevel>) -> LevelFilter {
-    match log_level {
-        None => LevelFilter::INFO,
-        Some(level) => match level {
-            LogLevel::Off => LevelFilter::OFF,
-            LogLevel::Error => LevelFilter::ERROR,
-            LogLevel::Warn => LevelFilter::WARN,
-            LogLevel::Info => LevelFilter::INFO,
-            LogLevel::Debug => LevelFilter::DEBUG,
-            LogLevel::Trace => LevelFilter::TRACE,
-        },
-    }
-}
-
 fn tracing_stdout_init(filter: LevelFilter, style: &TraceStyle) {
     let builder = tracing_subscriber::fmt().with_max_level(filter);
 

src/config/mod.rs

@@ -27,6 +27,7 @@ pub type Mail = v1::mail::Mail;
 pub type Network = v1::net::Network;
 pub type TrackerStatisticsImporter = v1::tracker_statistics_importer::TrackerStatisticsImporter;
 pub type Tracker = v1::tracker::Tracker;
+pub type Logging = v1::logging::Logging;
 pub type Website = v1::website::Website;
 pub type EmailOnSignup = v1::auth::EmailOnSignup;
 
@@ -325,7 +326,10 @@ mod tests {
 
     #[cfg(test)]
     fn default_config_toml() -> String {
-        let config = r#"[website]
+        let config = r#"[logging]
+                                log_level = "info"
+
+                                [website]
                                 name = "Torrust"
 
                                 [tracker]

src/config/v1/logging.rs

@@ -0,0 +1,76 @@
+use std::fmt;
+
+use serde::{Deserialize, Serialize};
+use tracing::level_filters::LevelFilter;
+
+/// Core configuration for the API
+#[derive(Debug, Clone, Serialize, Deserialize, PartialEq)]
+pub struct Logging {
+    /// Logging level. Possible values are: `Off`, `Error`, `Warn`, `Info`, `Debug`, `Trace`.
+    #[serde(default = "Logging::default_log_level")]
+    pub log_level: LogLevel,
+}
+
+impl Default for Logging {
+    fn default() -> Self {
+        Self {
+            log_level: Logging::default_log_level(),
+        }
+    }
+}
+
+impl Logging {
+    fn default_log_level() -> LogLevel {
+        LogLevel::Info
+    }
+}
+
+#[derive(Serialize, Deserialize, PartialEq, Eq, PartialOrd, Ord, Debug, Hash, Clone)]
+#[serde(rename_all = "lowercase")]
+pub enum LogLevel {
+    /// A level lower than all log levels.
+    Off,
+    /// Corresponds to the `Error` log level.
+    Error,
+    /// Corresponds to the `Warn` log level.
+    Warn,
+    /// Corresponds to the `Info` log level.
+    Info,
+    /// Corresponds to the `Debug` log level.
+    Debug,
+    /// Corresponds to the `Trace` log level.
+    Trace,
+}
+
+impl Default for LogLevel {
+    fn default() -> Self {
+        Self::Info
+    }
+}
+
+impl fmt::Display for LogLevel {
+    fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
+        let display_str = match self {
+            LogLevel::Off => "off",
+            LogLevel::Error => "error",
+            LogLevel::Warn => "warn",
+            LogLevel::Info => "info",
+            LogLevel::Debug => "debug",
+            LogLevel::Trace => "trace",
+        };
+        write!(f, "{display_str}")
+    }
+}
+
+impl From<LogLevel> for LevelFilter {
+    fn from(log_level: LogLevel) -> Self {
+        match log_level {
+            LogLevel::Off => LevelFilter::OFF,
+            LogLevel::Error => LevelFilter::ERROR,
+            LogLevel::Warn => LevelFilter::WARN,
+            LogLevel::Info => LevelFilter::INFO,
+            LogLevel::Debug => LevelFilter::DEBUG,
+            LogLevel::Trace => LevelFilter::TRACE,
+        }
+    }
+}