Skip to content

Commit 5364dc3

Browse files
fix: remove redundant part from alert notification message (#1439)
duplicate entries in message - 1. AlertName 2. Triggered TimeStamp 3. Severity
1 parent 4f68965 commit 5364dc3

File tree

2 files changed

+2
-12
lines changed

2 files changed

+2
-12
lines changed

src/alerts/alert_structs.rs

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -74,16 +74,6 @@ impl Context {
7474
}
7575
}
7676

77-
pub(crate) fn default_alert_string(&self) -> String {
78-
format!(
79-
"AlertName: {}\nTriggered TimeStamp: {}\nSeverity: {}\n{}",
80-
self.alert_info.alert_name,
81-
Utc::now().to_rfc3339(),
82-
self.alert_info.severity,
83-
self.message
84-
)
85-
}
86-
8777
pub(crate) fn default_resolved_string(&self) -> String {
8878
format!("{} is now `not-triggered` ", self.alert_info.alert_name)
8979
}

src/alerts/target.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -437,7 +437,7 @@ impl CallableTarget for SlackWebHook {
437437

438438
let alert = match payload.alert_info.alert_state {
439439
AlertState::Triggered => {
440-
serde_json::json!({ "text": payload.default_alert_string() })
440+
serde_json::json!({ "text": payload.message })
441441
}
442442
AlertState::NotTriggered => {
443443
serde_json::json!({ "text": payload.default_resolved_string() })
@@ -476,7 +476,7 @@ impl CallableTarget for OtherWebHook {
476476
.expect("Client can be constructed on this system");
477477

478478
let alert = match payload.alert_info.alert_state {
479-
AlertState::Triggered => payload.default_alert_string(),
479+
AlertState::Triggered => payload.message.clone(),
480480
AlertState::NotTriggered => payload.default_resolved_string(),
481481
AlertState::Disabled => payload.default_disabled_string(),
482482
};

0 commit comments

Comments
 (0)