exit with an error if we fail to parse the config file

just using the defaults is confusing for subcommands that don't display
errors by default (stream, record, etc)

1 files changed, 7 insertions, 5 deletions

diff --git a/src/cmd/play.rs b/src/cmd/play.rs
index 2e41816..7f6657b 100644
--- a/src/cmd/play.rs
+++ b/src/cmd/play.rs
@@ -27,11 +27,13 @@ pub fn cmd<'a, 'b>(app: clap::App<'a, 'b>) -> clap::App<'a, 'b> {
     crate::config::Ttyrec::cmd(app.about("Play recorded terminal sessions"))
 }
 
-pub fn config(config: config::Config) -> Box<dyn crate::config::Config> {
-    Box::new(config.try_into().unwrap_or_else(|e| {
-        log::warn!("failed to parse config data: {}", e);
-        Config::default()
-    }))
+pub fn config(
+    config: config::Config,
+) -> Result<Box<dyn crate::config::Config>> {
+    let config: Config = config
+        .try_into()
+        .context(crate::error::CouldntParseConfig)?;
+    Ok(Box::new(config))
 }
 
 #[allow(clippy::large_enum_variant)]