Add check for SES bounce notif type (#1123)

knadh · Jan 28, 2023 · 755344e · 755344e
1 parent 8985e5c
commit 755344e
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/internal/bounce/webhooks/ses.go b/internal/bounce/webhooks/ses.go
@@ -46,6 +46,7 @@ type sesNotif struct {
 type sesTimestamp time.Time
 
 type sesMail struct {
+	EventType string `json:"eventType"`
 	NotifType string `json:"notificationType"`
 	Bounce    struct {
 		BounceType string `json:"bounceType"`
@@ -118,6 +119,10 @@ func (s *SES) ProcessBounce(b []byte) (models.Bounce, error) {
 		return bounce, fmt.Errorf("error unmarshalling SES notification: %v", err)
 	}
 
+	if !(m.EventType == "Bounce" || m.NotifType == "Bounce") {
+		return bounce, errors.New("notification type is not bounce")
+	}
+
 	if len(m.Mail.Destination) == 0 {
 		return bounce, errors.New("no destination e-mails found in SES notification")
 	}