diff --git a/src/main.rs b/src/main.rs index 74901072..301aae62 100644 --- a/src/main.rs +++ b/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(()) } diff --git a/src/run.rs b/src/run.rs index 49deed94..0f32d392 100644 --- a/src/run.rs +++ b/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() )); }