Fix email notifier subject when nodes is empty

valeriansaliou · Aug 11, 2023 · 3ff07dc · 3ff07dc
1 parent da1993f
commit 3ff07dc
Showing 1 changed file with 9 additions and 5 deletions.
diff --git a/src/notifier/email.rs b/src/notifier/email.rs
@@ -65,11 +65,15 @@ impl GenericNotifier for EmailNotifier {
                     Some(APP_CONF.branding.page_title.to_owned()),
                     email_config.from.parse::<Address>().or(Err(true))?,
                 ))
-                .subject(format!(
-                    "{} | {}",
-                    notification.status.as_str().to_uppercase(),
-                    &nodes_label
-                ))
+                .subject(if nodes_label.is_empty() {
+                    notification.status.as_str().to_uppercase()
+                } else {
+                    format!(
+                        "{} | {}",
+                        notification.status.as_str().to_uppercase(),
+                        &nodes_label
+                    )
+                })
                 .body(message)
                 .or(Err(true))?;