Altinn / altinn-notifications

Altinn platform microservice for handling notifications (mail, sms, etc)
MIT License
2 stars 3 forks source link

Implement retry logic for intermittent errors in communication with Communication Services. #153

Closed acn-sbuad closed 10 months ago

acn-sbuad commented 1 year ago

Description

We already know of multiple potential faults that can occur when sending an email

We need to define a strategy for how to handle the various errors. Do we set a status code to retry after a given time frame? Do we stop attempting as the error indicates it will never work? etc.

useful doc: https://learn.microsoft.com/en-us/azure/communication-services/concepts/email/sender-reputation-managed-suppression-list

In scope

Constraints

Analysis

We have agreed on a data flow in the error situations. An additional topic for system errors or "system updates" will be created. Related microservices can use this topic to alert to instabilities or downtime in their system causing us to for instance pause the request for sending emails.

Tasks

Test cases

acn-sbuad commented 10 months ago

Regression tests suceessful. New feature tested ok in at22