Overriding defaultTitle with subject for SNS notifications

Signed-off-by: uros <[email protected]>
opensearch-project · Dec 20, 2022 · b7aab3b · b7aab3b
1 parent 817d9ef
commit b7aab3b
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/...src/main/kotlin/org/opensearch/alerting/util/destinationmigration/NotificationApiUtils.kt b/...src/main/kotlin/org/opensearch/alerting/util/destinationmigration/NotificationApiUtils.kt
@@ -138,12 +138,12 @@ suspend fun NotificationConfigInfo.sendNotification(client: Client, title: Strin
 }
 
 /**
- * For most channel types, a placeholder Alerting title will be used but the email channel will
- * use the subject, so it appears as the actual subject of the email.
+ * For most channel types, a placeholder Alerting title will be used but the email channel/SNS notification will
+ * use the subject, so it appears as the actual subject of the email/SNS notification.
  */
 fun NotificationConfigInfo.getTitle(subject: String?): String {
     val defaultTitle = "Alerting-Notification Action"
-    if (this.notificationConfig.configType == ConfigType.EMAIL) {
+    if (this.notificationConfig.configType == ConfigType.EMAIL || this.notificationConfig.configType == ConfigType.EMAIL) {
         return if (subject.isNullOrEmpty()) defaultTitle else subject
     }