fixes #31 allow slack alerts to be disabled

AppMetrics · Jul 9, 2018 · 1175a20 · 1175a20
1 parent b540a74
commit 1175a20
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/src/App.Metrics.Health.Reporting.Slack/SlackHealthAlertOptions.cs b/src/App.Metrics.Health.Reporting.Slack/SlackHealthAlertOptions.cs
@@ -15,5 +15,7 @@ public class SlackHealthAlertOptions
         public string EmojiIcon { get; set; }
 
         public bool AlertOnDegradedChecks { get; set; } = true;
+
+        public bool Enabled { get; set; } = true;
     }
 }
diff --git a/src/App.Metrics.Health.Reporting.Slack/SlackIncomingWebHookHealthAlerter.cs b/src/App.Metrics.Health.Reporting.Slack/SlackIncomingWebHookHealthAlerter.cs
@@ -33,6 +33,11 @@ public SlackIncomingWebHookHealthAlerter(SlackHealthAlertOptions slackOptions)
         /// <inheritdoc />
         public async Task ReportAsync(HealthOptions options, HealthStatus status, CancellationToken cancellationToken = default)
         {
+            if (!_slackOptions.Enabled)
+            {
+                return;
+            }
+
             var applicationName = options.ApplicationName;
 
             if (Uri.TryCreate(applicationName, UriKind.Absolute, out var appUri))