Development #43496
ne pas utiliser noreply@entrouvert.com en From par défaut
0%
Description
if not email_from: email_from = emails_cfg.get('from', 'noreply@entrouvert.com')
qui est un truc fait depuis le tout début; mais avec @entrouvert.com désormais sous DKIM/DMARC/etc., on ne peut plus trop avoir ça comme comportement par défaut, je pense.
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a presque 4 ans
Ma proposition serait très brutale : ne pas envoyer de mail tant que le from n'est pas configuré.
--- a/wcs/qommon/emails.py +++ b/wcs/qommon/emails.py @@ -171,6 +171,11 @@ def email(subject, mail_body, email_rcpt, replyto=None, bcc=None, emails_cfg = get_cfg('emails', {}) footer = emails_cfg.get('footer') or '' + if not email_from: + email_from = emails_cfg.get('from') + if not email_from: + return + encoding = get_publisher().site_charset
Mis à jour par Frédéric Péters il y a presque 4 ans
- Fichier 0001-emails-compute-default-from-using-local-user-hostnam.patch 0001-emails-compute-default-from-using-local-user-hostnam.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Je trouve ça assez excessif (vu que ce n'est pas configuré sur certains sites, et que ça signifierait ne pas recevoir de mail); j'imaginais plutôt poser un qqch genre
pwd.getpwuid(os.getuid())[0] + "@" + socket.getfqdn()
voilà, patch attaché.
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
Pour les trucs hébergés hors de chez nous, ça ira et ça évite de pourrir le score de notre domaine (si ça a un sens).
Par contre pour le SaaS est-ce que ça aura un impact sur nos scores d'envoyer des mails en authentic/wcs.nodeX.prod.saas.entrouvert.org
?
Mis à jour par Frédéric Péters il y a presque 4 ans
Personne ne sait comment ces systèmes fonctionnent, on ne peut pas répondre à cette question.
Mais on peut assurer que nos systèmes continuent à se déployer avec noreply@entrouvert.com en From, il suffit de modifier les fichiers modèles de site. (j'ai fait ça).
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
- Statut changé de Solution proposée à Solution validée
Allez hop, ok.
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 9f0160549c6ef6773cb6edca4cd5ae0cc7b92fc8 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Mon Jun 1 09:23:25 2020 +0200 emails: compute default from using local user/hostname (#43496)
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
emails: compute default from using local user/hostname (#43496)