Remove Option from slack_webhook

fpco · Jul 25, 2024 · 0ea46af · 0ea46af
1 parent 232be30
commit 0ea46af
Showing 1 changed file with 16 additions and 18 deletions.
diff --git a/src/cli.rs b/src/cli.rs
@@ -58,7 +58,7 @@ pub(crate) struct Cli {
 #[group(required = true, multiple = false)]
 pub(crate) struct NotifyHook {
     #[arg(long, value_parser(Url::from_str), env = "HEALTH_CHECK_SLACK_WEBHOOK")]
-    pub(crate) slack_webhook: Option<Url>,
+    pub(crate) slack_webhook: Url,
 }
 
 #[derive(Debug)]
@@ -191,23 +191,21 @@ impl Cli {
         match res {
             Ok(()) => Ok(()),
             Err(e) => {
-                if let Some(slack_webhook) = self.notification_hook.slack_webhook {
-                    let slack_app = SlackApp::new(
-                        slack_webhook,
-                        self.notification_context,
-                        self.app_description,
-                        self.app_version,
-                        self.image_url,
-                    );
-                    let mut msg = String::new();
-                    for line in &*recent_output.lock() {
-                        msg.push_str(line);
-                        msg.push('\n');
-                    }
-                    let result = slack_app.send_notification(&e, &msg);
-                    if let Err(err) = result {
-                        eprintln!("Slack notification failed: {err:?}");
-                    }
+                let slack_app = SlackApp::new(
+                    self.notification_hook.slack_webhook,
+                    self.notification_context,
+                    self.app_description,
+                    self.app_version,
+                    self.image_url,
+                );
+                let mut msg = String::new();
+                for line in &*recent_output.lock() {
+                    msg.push_str(line);
+                    msg.push('\n');
+                }
+                let result = slack_app.send_notification(&e, &msg);
+                if let Err(err) = result {
+                    eprintln!("Slack notification failed: {err:?}");
                 }
                 Err(e)
             }