[Tech - email] Système de secours en cas d'erreur d'envoi

[hmeneuvrier]

Ticket

#3260

Description

Stockage en base des mails non-envoyés (erreurs) et création d'une commande pour les renvoyer

Changements apportés

• Création de la table failed_email (migrations/entité)
• Création d'une commande retry-failed-emails pour l'instant à lancer à la main de temps en temps
• Modification du AbstractNotificationMailer pour enregistrer les mails qui ne sont pas envoyés dans la table (sauf si on passe par la commande)
• Mise à jour de tous les service/mailer pour passer entityManager
• Mise à jour ActivityListener pour enregistrer le suivi concerné dans une colonne à part entière et pas dans les params (et mise à jour en conséquence de SuiviNewCommentBackMailer, du twig associé et de NotificationMail)
• Mise à jour de tous les templates de mails (et les services/mailers associés) pour ne pas utiliser directement d'entités dans les templates.

Pré-requis

make execute-migration name=Version20241112145634 direction=up

Dans AbstractNotificationMailer remplacer les lignes 96 à 102

        try {
            $this->mailer->send($message);

            return true;
        } catch (\Throwable $exception) {
            $this->logAndSaveFailedEmail($message, $notificationMail, $exception, $params, $saveFailedMail);
        }

par

        $exception = new \Exception(message: 'test');
        $this->logAndSaveFailedEmail($message, $notificationMail, $exception, $params, $saveFailedMail);

Tests

• Faire des actions sur la plateforme pour générer le plus de mails possibles (notamment des mails d'ajout de suivi, des mails avec pièce jointe -> nouveau signalement de locataire avec proprio pas averti)
• Vérifier le remplissage de la table failed_email
• Lancer la commande make console app="retry-failed-emails" et vérifier qu'il n'y a pas d'erreur, et que les mails sont bien renvoyés
• Remettre AbstractNotificationMailer en état

[emilschn]

Mes tests semblent concluants :)

[hmeneuvrier]

du coup 3 trucs à voir avec Saidi demain matin

• faire un manager et un rerpo pour FailedEmail ?
• l'injection de setter pour entityManager dans AbstractNotificationMailer
• boucle infinie sur ActivityListener (faut que je creuse un peu plus, mais là, je fatigue)

[numew]

Test ok,
Dommage de ne plus pouvoir passer des object en parametre aux template d'email et les changements que ca implique mais bon si y'a pas d'autre solutions simple ca me va

[sonarcloud]

Quality Gate failed

Failed conditions
14.7% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

[numew]

ok

[sfinx13]

Lecture et test OK
Des les premiers erreurs faudra déclarer un cron pour plus s'en soucier toutes les heures ou 2 heures (je crée le ticket)
Lecture et test OK :-)