New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

En-tête d'e-mail "X-Mailer" dédiée #80

Closed
C-Duv opened this Issue Sep 3, 2017 · 4 comments

Comments

Projects
None yet
2 participants
@C-Duv
Contributor

C-Duv commented Sep 3, 2017

Je trouverais pratique que les e-mails émis par Cheky aient une en-tête dédiée, ça permet de simplifier ses filtres/règles de classement d'e-mail sur son client de messagerie.

Actuellement, les e-mails émis par Cheky ont l'en-tête suivante :
X-Mailer: PHPMailer 5.2.7 (https://github.com/PHPMailer/PHPMailer/)

Je propose quelque chose comme :
X-Mailer: Cheky

Il peut être intéressant d'avoir la version, on peut aussi ajouter l'en-tête X-Mailer-Version telle qu'on aurait alors :

X-Mailer: Cheky
X-Mailer-Version: 3.6.1

(La version permet notamment de débuguer plus facilement lors qu'un problème est signalé sur un e-mail)

La librairie utilisée (PHPMailer) le permet ainsi :

$mail->addCustomHeader('X-Mailer', 'Cheky');
$mail->addCustomHeader('X-Mailer-Version ', APPLICATION_VERSION);

A rajouter vers les lignes 55-57 de /app/notifier/scripts/check.php par exemple : https://github.com/Blount/LBCAlerte/blob/22e26b5d467825e03f9d02fd94aac5a92490a230/app/notifier/scripts/check.php#L55-L57

@Blount

This comment has been minimized.

Owner

Blount commented Sep 4, 2017

Oui, c'est une idée intéressante. Après, je ne sais pas dans quel mesure est utilisé l'entête X-Mailer pour la détection de spam. Mais on peut essayer.
Sinon, dans la même logique, on pourrait ajouter une entête pour spécifier la catégorie de l'alerte (X-Cheky-Category), ce qui permettrait de faciliter les tris dans la boîte mail.

@C-Duv

This comment has been minimized.

Contributor

C-Duv commented Sep 8, 2017

Généralement, X-Mailer est assez libre.
Tu as raison pour les catégorie : c'est typiquement ce que fait Redmine.
GitHub utilise aussi des en-tête dédiée.

  • X-Mailer: Redmine
  • X-Redmine-Project; Foobar
  • X-Redmine-Issue-Id: 42
  • X-GitHub-Reason: mention

@Blount Blount closed this in 8d55fb9 Sep 14, 2017

@C-Duv

This comment has been minimized.

Contributor

C-Duv commented Aug 1, 2018

Au fait, je n'avais pas pris le temps de te remercier d'avoir intégré ma proposition : Merci 👍

@Blount

This comment has been minimized.

Owner

Blount commented Aug 29, 2018

De rien ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment