matthiasbeyer/imag
1
0
Fork 0
Code Releases Activity

Fix: --debug on CLI should enable logging and override it

Browse source 
When specifying the `--debug` flag on the commandline, logging was not
enabled. That was because the config file parsing did not consider the
args.

Now, if `--debug` is passed on the CLI, logging is enabled for all
modules and level is set to `debug` for all modules.

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
This commit is contained in:
Matthias Beyer 2018-10-26 15:41:00 +02:00
parent ca9905c6fb
commit 22c13fed42
1 changed files with 24 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

19
lib/core/libimagrt/src/logger.rs
View file

@ -39,6 +39,7 @@ use handlebars::Handlebars;
type ModuleName = String; type ModuleName = String;
type Result<T> = ::std::result::Result<T, RE>; type Result<T> = ::std::result::Result<T, RE>;
#[derive(Debug)]
enum LogDestination { enum LogDestination {
Stderr, Stderr,
File(Arc<Mutex<::std::fs::File>>), File(Arc<Mutex<::std::fs::File>>),
@ -50,6 +51,7 @@ impl Default for LogDestination {
} }
} }
#[derive(Debug)]
struct ModuleSettings { struct ModuleSettings {
enabled: bool, enabled: bool,
level: Option<Level>, level: Option<Level>,
@ -106,10 +108,13 @@ impl ImagLogger {
handlebars.register_template_string("ERROR", fmt)?; // name must be uppercase handlebars.register_template_string("ERROR", fmt)?; // name must be uppercase
} }
let module_settings = aggregate_module_settings(matches, config)?;
eprintln!("Logging: {:?}", module_settings);
Ok(ImagLogger { Ok(ImagLogger {
global_loglevel : aggregate_global_loglevel(matches, config)?, global_loglevel : aggregate_global_loglevel(matches, config)?,
global_destinations : aggregate_global_destinations(matches, config)?, global_destinations : aggregate_global_destinations(matches, config)?,
module_settings : aggregate_module_settings(matches, config)?, module_settings : module_settings,
handlebars : handlebars, handlebars : handlebars,
}) })
} }
@ -372,7 +377,7 @@ fn aggregate_global_format_error(config: Option<&Value>)
config) config)
} }
fn aggregate_module_settings(_matches: &ArgMatches, config: Option<&Value>) fn aggregate_module_settings(matches: &ArgMatches, config: Option<&Value>)
-> Result<BTreeMap<ModuleName, ModuleSettings>> -> Result<BTreeMap<ModuleName, ModuleSettings>>
{ {
// Helper macro to return the error from Some(Err(_)) and map everything else to an // Helper macro to return the error from Some(Err(_)) and map everything else to an
@ -407,11 +412,19 @@ fn aggregate_module_settings(_matches: &ArgMatches, config: Option<&Value>)
}) })
}; };
let (pre_enabled, level) = if matches.is_present(Runtime::arg_debugging_name()) {
(true, Some(Level::Debug))
} else {
let level = inner_try! { let level = inner_try! {
v.read_string("level")?.map(|s| match_log_level_str(&s)) v.read_string("level")?.map(|s| match_log_level_str(&s))
}; };
let enabled = v.read("enabled")? (false, level)
};
let enabled = pre_enabled ||
v.read("enabled")?
.map(|v| v.as_bool().unwrap_or(false)) .map(|v| v.as_bool().unwrap_or(false))
.ok_or_else(|| { .ok_or_else(|| {
let path = "imag.logging.modules.<mod>.enabled".to_owned(); let path = "imag.logging.modules.<mod>.enabled".to_owned();