Projet

Général

Profil

Development #43496

ne pas utiliser noreply@entrouvert.com en From par défaut

Ajouté par Frédéric Péters il y a presque 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
31 mai 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

Révision 9f016054 (diff)
Ajouté par Frédéric Péters il y a presque 4 ans

emails: compute default from using local user/hostname (#43496)

Historique

#1

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

#2

Mis à jour par Frédéric Péters il y a presque 4 ans

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é.

#3

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 ?

#4

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 en From, il suffit de modifier les fichiers modèles de site. (j'ai fait ça).

#5

Mis à jour par Benjamin Dauvergne il y a presque 4 ans

  • Statut changé de Solution proposée à Solution validée

Allez hop, ok.

#6

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)
#7

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

Formats disponibles : Atom PDF