Merge 4de15eda66 into 280c7e2b5e

src/config/json.rs

@@ -4,7 +4,7 @@
 use std::fs::File;
 use std::io::BufReader;
 
-use crate::config::{process_settings, set_logging_level, Config};
+use crate::config::{process_settings, Config};
 use crate::handler::{file_path, FileType};
 
 impl Config {
@@ -20,7 +20,11 @@ impl Config {
             Ok(f) => f,
             Err(_) => {
                 log::error!("Config Error: No config file found, falling back to defaults");
-                return Ok(Self::default());
+                let conf = Self::default();
+                if !logging_initialized {
+                    conf.set_logging_level();
+                }
+                return Ok(conf);
             }
         };
 
@@ -29,7 +33,7 @@ impl Config {
         let mut conf: Config = serde_json::from_reader(reader)?;
 
         if !logging_initialized {
-            set_logging_level(conf.server.debug, conf.server.logging)
+            conf.set_logging_level();
         }
 
         conf = process_settings(conf)?;

src/config/lua.rs

@@ -4,7 +4,7 @@ use std::fs;
 
 use mlua::{Lua, LuaSerdeExt};
 
-use crate::config::{process_settings, set_logging_level, Config};
+use crate::config::{process_settings, Config};
 use crate::handler::{file_path, FileType};
 
 impl Config {
@@ -21,7 +21,11 @@ impl Config {
             Ok(f) => f,
             Err(_) => {
                 log::error!("Config Error: No config file found, falling back to defaults");
-                return Ok(Self::default());
+                let conf = Self::default();
+                if !logging_initialized {
+                    conf.set_logging_level();
+                }
+                return Ok(conf);
             }
         };
         let val = lua.load(fs::read_to_string(config_file)?).eval()?;
@@ -29,7 +33,7 @@ impl Config {
         let mut conf: Config = lua.from_value(val)?;
 
         if !logging_initialized {
-            set_logging_level(conf.server.debug, conf.server.logging)
+            conf.set_logging_level();
         }
 
         conf = process_settings(conf)?;

src/config/mod.rs

@@ -20,13 +20,9 @@ pub struct Config {
     pub search: Search,
 }
 
-/// a helper function that sets the proper logging level
+impl Config {
-///
+    /// a helper function that sets the proper logging level
-/// # Arguments
+    fn set_logging_level(&self) {
-///
-/// * `debug` - It takes the option to whether enable or disable debug mode.
-/// * `logging` - It takes the option to whether enable or disable logs.
-fn set_logging_level(debug: bool, logging: bool) {
         if let Ok(pkg_env_var) = std::env::var("PKG_ENV") {
             if pkg_env_var.to_lowercase() == "dev" {
                 env_logger::Builder::new()
@@ -37,7 +33,7 @@ fn set_logging_level(debug: bool, logging: bool) {
         }
 
         // Initializing logging middleware with level set to default or info.
-    let log_level = match (debug, logging) {
+        let log_level = match (self.server.debug, self.server.logging) {
             (true, true) => LevelFilter::Debug,
             (true, false) => LevelFilter::Debug,
             (false, true) => LevelFilter::Info,
@@ -45,6 +41,7 @@ fn set_logging_level(debug: bool, logging: bool) {
         };
 
         env_logger::Builder::new().filter(None, log_level).init();
+    }
 }
 
 fn process_settings(mut conf: Config) -> Result<Config, Box<dyn std::error::Error>> {