Backlog Courrier

Message de Florent Cappon sur le ticket client

Bonjour Henri, Emmanuel,

Essayant de vous fournir les logs appropriés je suis tombé dans des problèmes de non-remontée de ces derniers et je me suis lancé dans le debug du souci qui m'a permis de finir par identifier la cause du souci initial : elle se situe à la ligne 758 du fichier src/app/email/controllers/EmailController.php.

Voici le bout de code en question :

757 // If we cannot override from with the sender email address, we try sending the email with the from in the configuration
758 if (strpos($errors, 'Client does not have permissions to send as this sender') !== false && $email['sender']['email'] != $configuration['from']) {
759 $sender = [
760   'email'    => $configuration['from'],
761   'entityId' => $email['sender']['entityId'] ?? null
762 ];
763 EmailModel::update(['set' => ['sender' => json_encode($sender)], 'where' => ['id = ?'], 'data' => [$args['emailId']]]);
764
765 return EmailController::sendEmail(['emailId' => $args['emailId'], 'userId' => $args['userId']]);
766 }
767
768 return ['errors' => $errors];

Il permet d'appliquer la bascule "automatique" vers l'adresse d'envoi renseignée dans l'administration, comme l'a mentionné Emmanuel, sauf qu'elle est conditionnée à deux clauses :

• la non correspondance entre la première adresse testée et l'adresse renseignée dans l'administration => OK,

et

• la correspondance entre le message d'erreur remonté par phpmailer et "Client does not have permissions to send as this sender" => KO, c'est cette condition qui fait échouer le test et empêche la bascule sur l'adresse d'administration

En l'occurence notre smtp renvoie l'erreur suivante via phpmailer :

SMTP ERROR: RCPT TO command failed: 553 5.7.1 <florent.cappon@developpement-durable.gouv.fr>: Sender address rejected: not logged in

J'imagine que cette condition a été implémentée initialement par précaution après un test sur votre smtp de dév qui doit renvoyer l'erreur précise que vous avez mis en dur dans le code.

Ma conclusion : enlever cette condition suffit à rétablir la bascule automatique et elle ne semble pas être nécessaire de toute manière.
Cette modification peut-elle être intégrée à un tag dès que possible ?

Bien cordialement,

P.-S. : coder des valeurs en dur c'est le Mal !

Ajouter le test sur "Sender address rejected" dans la mécanique de bascule.

TEST 21.03 / Develop

MAJ Branche Develop->2301