Accepting AWS SNS notifications (Delivery, Send, ...) as Bounce

[hasahmad]

Version:

• listmonk: v2.3.0
• OS: Docker Image

Description of the bug and steps to reproduce:
I initially setup in AWS SNS to send all even types (Delivery, Send, ...) and I noticed in listmonk Bounces list that emails with delivery status were also marked as bounce.
I then changed to only send notification for Bounces.

In the code I don't see any checks for eventType, I think we should add check in code to make sure the eventType is Bounce (and maybe also Complaint?)
In sesMail struct, I see notificationType but not eventType where I noticed the Bounce event type.

Payload for notificationType = Delivery

{
  "mail": { ... },
  "delivery": { ... },
  "notificationType": "Delivery"
}

Payload for eventType = Bounce

{
  "mail": { ... },
  "bounce": { ... },
  "eventType": "Bounce"
}

[knadh]

Will add this check, but non-bounce notifications shouldn't be sent to listmonk. I recollect that when configuring SNS notifications, it's possible to enable bounce-only events.

[kisero]

In order to improve bounce management it would be great to have an additional column on the schema for bounces that describes the bounce type: Permanent, Transient, Etc.

Discourse has the following settings for bounce type management perhaps this helps for some insipiration. From what I've seen listmonk does this in a slight more limited way.

[knadh]

5fc28a7 adds support for per-bounce-type actions.