Change where logging is init

- Needed to change because going to using current working directory to
    decide where to put logs thus need that to be set before starting
    logging.
- So removed log messages during folder setting as they cannot be seen
    and would just be misleading.
- Added new log message after setting current working directory. Also
    marks the start of the program.

src/cli.rs

@@ -1,8 +1,8 @@
-use std::{env, path::PathBuf};
+use std::path::PathBuf;
 
 use anyhow::Context;
 use clap::{Parser, ValueEnum};
-use log::{debug, info, LevelFilter};
+use log::LevelFilter;
 
 #[derive(Parser, Clone, Eq, PartialEq, Ord, PartialOrd, Debug, Default)]
 #[command(
@@ -29,20 +29,9 @@ impl Cli {
     }
     /// Changes the current working directory to path if one is given
     pub fn update_current_working_dir(&self) -> anyhow::Result<()> {
-        debug!(
-            "Before attempting update current dir, it is: {}",
-            env::current_dir()?.display()
-        );
         if let Some(path) = &self.working_dir {
-            info!("Going to update working directory to to '{path}'");
             std::env::set_current_dir(path)
                 .with_context(|| format!("Failed to set current dir to: '{path}'"))?;
-            info!(
-                "After updating current dir, it is: '{}'",
-                env::current_dir()?.display()
-            );
-        } else {
-            debug!("No user supplied path found. No change")
         }
         Ok(())
     }

src/lib.rs

@@ -2,6 +2,7 @@
 mod cli;
 mod config;
 mod event_recorder;
+mod logging;
 mod notification;
 mod ping;
 mod state_management;
@@ -22,13 +23,20 @@ pub(crate) use crate::{
 };
 use anyhow::Context;
 use event_recorder::{ResponseMessage, TargetID};
-use log::debug;
+use log::{debug, warn};
 
 pub use crate::{cli::Cli, event_recorder::TimestampedResponse};
 
 pub fn run(cli: Cli) -> anyhow::Result<()> {
     cli.update_current_working_dir()
         .context("Failed to update current working directory")?;
+    logging::init_logging(cli.log_level.into())?;
+    warn!(
+        "Starting up in dir: {:?}",
+        std::env::current_dir()
+            .context("Failed to get cwd")?
+            .display()
+    );
     let config = Config::load_from(&cli.get_config_path()).context("Failed to load config")?;
 
     let (tx, rx) = mpsc::channel();

src/main.rs

@@ -1,17 +1,8 @@
 use clap::Parser;
 use conn_mon::{run, Cli};
-use env_logger::Builder;
-use log::LevelFilter;
 
 fn main() -> anyhow::Result<()> {
     let cli = Cli::parse();
-    init_logging(cli.log_level.into())?;
     run(cli)?;
     Ok(())
 }
-
-fn init_logging(level: LevelFilter) -> anyhow::Result<()> {
-    // TODO: Change to log to file
-    Builder::new().filter(None, level).try_init()?;
-    Ok(())
-}