cli: improve readability of application output

Cargo.toml

@@ -14,7 +14,7 @@ chrono = "0.4"
 dbus = { version = "0.9", optional = true }
 dbus-tokio = { version = "0.7.3", optional = true }
 dbus-crossroads = { version = "0.5.0", optional = true }
-fern = { version = "0.7.0", features = ["syslog-6"] }
+fern = { version = "0.7.0", features = ["syslog-6", "colored"] }
 futures = "0.3.15"
 gethostname = "0.5.0"
 hex = "0.4"

src/config.rs

@@ -207,7 +207,7 @@ where
         toml::Value::String(num) => {
             return u8::from_str(&num)
                 .map(Some)
-                .inspect(|_| warn!("Configuration Warning: `initial_volume` should be a number rather than a string, this will become a hard error in the future"))
+                .inspect(|_| warn!("`initial_volume` should be a number rather than a string, this will become a hard error in the future"))
                 .map_err(de::Error::custom)
         }
         toml::Value::Float(f) => Unexpected::Float(f),
@@ -484,14 +484,14 @@ impl CliConfig {
             if let Some(problem) = get_known_config_problem(&path) {
                 match problem {
                     KnownConfigProblem::MissingFeature(feature) => {
-                        warn!("Warning: The config key '{path}' is ignored, because the feature '{feature}' is missing in this build");
+                        warn!("The config key '{path}' is ignored, because the feature '{feature}' is missing in this build");
                     }
                     KnownConfigProblem::UsernamePassword => {
-                        warn!("Warning: Authentication via Username and Password is no longer supported by Spotify, use ```spotifyd authenticate` instead");
+                        warn!("The config key '{path}' is ignored, because authentication with username and password is no longer supported by Spotify. Please use `spotifyd authenticate` instead");
                     }
                 }
             } else {
-                warn!("Warning: Unknown key '{path}' in config will be ignored");
+                warn!("Unknown key '{path}' in config will be ignored");
             }
         })?;
 

src/main.rs

@@ -6,6 +6,7 @@ use color_eyre::eyre::{self, Context};
 use config::ExecutionMode;
 #[cfg(unix)]
 use daemonize::Daemonize;
+use fern::colors::ColoredLevelConfig;
 #[cfg(unix)]
 use log::error;
 use log::{info, trace, LevelFilter};
@@ -40,23 +41,32 @@ fn setup_logger(log_target: LogTarget, verbose: u8) -> eyre::Result<()> {
         1 => LevelFilter::Debug,
         2.. => LevelFilter::Trace,
     };
+
     let mut logger = fern::Dispatch::new().level(log_level);
 
-    logger = if verbose > 0 {
-        logger.format(|out, message, record| {
-            out.finish(format_args!(
-                "[{} {}] {}",
-                record.level(),
-                record.target(),
-                message
-            ))
-        })
-    } else {
-        logger.level_for("symphonia_format_ogg::demuxer", LevelFilter::Warn)
-    };
+    if verbose == 0 {
+        logger = logger.level_for("symphonia_format_ogg::demuxer", LevelFilter::Warn);
+    }
 
     let logger = match log_target {
-        LogTarget::Terminal => logger.chain(std::io::stdout()),
+        LogTarget::Terminal => {
+            let colors = ColoredLevelConfig::new();
+            logger
+                .format(move |out, message, record| {
+                    let target = if verbose > 0 {
+                        &format!(" {}", record.target())
+                    } else {
+                        ""
+                    };
+                    out.finish(format_args!(
+                        "[{}{}] {}",
+                        colors.color(record.level()),
+                        target,
+                        message
+                    ))
+                })
+                .chain(std::io::stdout())
+        }
         #[cfg(unix)]
         LogTarget::Syslog => {
             let log_format = syslog::Formatter3164 {