brunob
commented
2 years ago Ok, donc, la classe PHPMailer semble ajouter par défaut un Message-ID ici https://git.spip.net/spip-contrib-extensions/facteur/src/tag/v4.1.1/lib/PHPMailer-6/src/PHPMailer.php#L2402 ce qui donne ceci dans $result à ce moment dans le code :
Date: Fri, 8 Apr 2022 19:45:37 +0200
To: xx@xx.fr
From: "anna doll (anna) - SeenThis" <robot@octopouce.net>
Subject: =?utf-8?Q?Re_:_Encore_une_image_qui_va_se_faire_bloquer..._..._Mais_pas_c?=
=?utf-8?Q?elle_l=C3=A0_!?=
Message-ID: <@localhost>Puis ensuite, la fonction ajoute le message-id qu'on passe depuis https://github.com/seenthis/seenthis/blob/master/inc/seenthis_notifier.php#L73 et ça donne bien un double message-di dans le $result :
"Date: Fri, 8 Apr 2022 19:45:37 +0200
To: xx@xx.fr
From: "anna doll (anna) - SeenThis" <robot@octopouce.net>
Subject: =?utf-8?Q?Re_:_Encore_une_image_qui_va_se_faire_bloquer..._..._Mais_pas_c?=
=?utf-8?Q?elle_l=C3=A0_!?=
Message-ID: <@localhost>
X-Mailer: PHPMailer 6.1.3 (https://github.com/PHPMailer/PHPMailer)
Message-Id: <829.b0363dbeb1f99466e462201cd15f0078@localhost/seenthis>
In-Reply-To: <794@localhost/seenthis>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8bitMême en remplaçant tous les Message-Id de seenthis_notifier par Message-ID, le résultat est le même. Je pense la classe la manière dont on utilise la classe PHPMailer dans inc_envoyer_mail() du plugin facteur ne permet pas de spécifier un Message-ID (en tout cas ce header ne doit pas être passé dans les CustomHeader).
En ajoutant une exception dans inc_envoyer_mail') à ce niveau https://git.spip.net/spip-contrib-extensions/facteur/src/tag/v4.1.1/inc/envoyer_mail.php#L328 comme ceci, c'est bon, le header perso pour message id est bien posé comme il faut :
if (!empty($headers)) {
foreach($headers as $h){
// verifions le format correct : il faut au moins un ":" dans le header
// et on filtre le Content-Type: qui sera de toute facon fourni par facteur
if (strpos($h,":")!==false
AND strncmp($h,"Content-Type:",13)!==0) {
if (strpos($h,"Message-ID:")===0) {
$facteur->MessageID = trim(explode(':', $h, 2)[1]);
} else {
$facteur->AddCustomHeader($h);
}
}
}
}Maintenant, reste à voir comment proposer le patch dans le plugin facteur, qui a bien bougé entre la version qu'on utilise et la dernière en date... (sur ce, j'ai faim ^^)
Pour le contexte https://discuter.rezo.net/t/anomalie-double-header-message-id/33871/3