From 305a6a0d8ba35d9b72cd2ffa9e2d3a7655f8112d Mon Sep 17 00:00:00 2001
From: Nikhil Sinha <nikhil.sinha@parseable.com>
Date: Tue, 23 Sep 2025 23:11:50 -0700
Subject: [PATCH] fix: remove redundant part from alert notification message

duplicate entries in message -
1. AlertName
2. Triggered TimeStamp
3. Severity
---
 src/alerts/alert_structs.rs | 10 ----------
 src/alerts/target.rs        |  4 ++--
 2 files changed, 2 insertions(+), 12 deletions(-)

diff --git a/src/alerts/alert_structs.rs b/src/alerts/alert_structs.rs
index f7d27d15f..5b60a6150 100644
--- a/src/alerts/alert_structs.rs
+++ b/src/alerts/alert_structs.rs
@@ -74,16 +74,6 @@ impl Context {
         }
     }
 
-    pub(crate) fn default_alert_string(&self) -> String {
-        format!(
-            "AlertName: {}\nTriggered TimeStamp: {}\nSeverity: {}\n{}",
-            self.alert_info.alert_name,
-            Utc::now().to_rfc3339(),
-            self.alert_info.severity,
-            self.message
-        )
-    }
-
     pub(crate) fn default_resolved_string(&self) -> String {
         format!("{} is now `not-triggered` ", self.alert_info.alert_name)
     }
diff --git a/src/alerts/target.rs b/src/alerts/target.rs
index 3ec9c2005..6c1eb9b42 100644
--- a/src/alerts/target.rs
+++ b/src/alerts/target.rs
@@ -437,7 +437,7 @@ impl CallableTarget for SlackWebHook {
 
         let alert = match payload.alert_info.alert_state {
             AlertState::Triggered => {
-                serde_json::json!({ "text": payload.default_alert_string() })
+                serde_json::json!({ "text": payload.message })
             }
             AlertState::NotTriggered => {
                 serde_json::json!({ "text": payload.default_resolved_string() })
@@ -476,7 +476,7 @@ impl CallableTarget for OtherWebHook {
             .expect("Client can be constructed on this system");
 
         let alert = match payload.alert_info.alert_state {
-            AlertState::Triggered => payload.default_alert_string(),
+            AlertState::Triggered => payload.message.clone(),
             AlertState::NotTriggered => payload.default_resolved_string(),
             AlertState::Disabled => payload.default_disabled_string(),
         };