fix: empty reply to address from env

PeerDB-io · Mar 12, 2024 · 25c6777 · 25c6777
1 parent f4a3968
commit 25c6777
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/flow/alerting/alerting.go b/flow/alerting/alerting.go
@@ -52,10 +52,15 @@ func (a *Alerter) registerSendersFromPool(ctx context.Context) ([]AlertSenderCon
 
 			alertSenderConfigs = append(alertSenderConfigs, AlertSenderConfig{Id: id, Sender: newSlackAlertSender(&slackServiceConfig)})
 		case EMAIL:
+			var replyToAddresses []string
+			if envReplyToAddresses := strings.Split(peerdbenv.PeerDBAlertingEmailSenderReplyToAddresses(), ","); len(envReplyToAddresses) != 0 {
+				// AWS SDK does not like empty slice
+				replyToAddresses = envReplyToAddresses
+			}
 			emailServiceConfig := EmailAlertSenderConfig{
 				sourceEmail:          peerdbenv.PeerDBAlertingEmailSenderSourceEmail(),
 				configurationSetName: peerdbenv.PeerDBAlertingEmailSenderConfigurationSet(),
-				replyToAddresses:     strings.Split(peerdbenv.PeerDBAlertingEmailSenderReplyToAddresses(), ","),
+				replyToAddresses:     replyToAddresses,
 			}
 			if emailServiceConfig.sourceEmail == "" {
 				return errors.New("missing sourceEmail for Email alerting service")