Merge pull request #837 from jvanz/better-error-message

fix: improve error handling in settings validation.

src/main.rs

@@ -247,7 +247,15 @@ async fn main() -> Result<()> {
         Some("run") => {
             if let Some(matches) = matches.subcommand_matches("run") {
                 let pull_and_run_settings = parse_pull_and_run_settings(matches).await?;
-                run::pull_and_run(&pull_and_run_settings).await?;
+                run::pull_and_run(&pull_and_run_settings)
+                    .await
+                    .map_err(|e| {
+                        anyhow!(
+                            "Error running policy {}: {}",
+                            pull_and_run_settings.uri,
+                            e.to_string()
+                        )
+                    })?;
             }
             Ok(())
         }

src/run.rs

@@ -168,10 +168,9 @@ pub(crate) async fn pull_and_run(cfg: &PullAndRunSettings) -> Result<()> {
     // validate the settings given by the user
     let settings_validation_response = policy_evaluator.validate_settings(&run_env.policy_settings);
     if !settings_validation_response.valid {
-        println!("{}", serde_json::to_string(&settings_validation_response)?);
         return Err(anyhow!(
             "Provided settings are not valid: {:?}",
-            settings_validation_response.message
+            settings_validation_response.message.unwrap_or_default()
         ));
     }