Projet

Général

Profil

Development #33437

utiliser le titre du site comme libellé pour l'adresse "From" des mails

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

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Dans w.c.s. on pose l'adresse "nue", dans les autres modules par contre l'adresse apparait "global title <email>".


Fichiers

Révisions associées

Révision 94c00fbe (diff)
Ajouté par Frédéric Péters il y a presque 5 ans

emails: use global title in From header (#33437)

Historique

#2

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

  • Assigné à mis à Frédéric Péters
#3

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

Dans "site_name" on a le nom posé sur le service dans hobo et ce n'est pas ça qu'on veut, on veut le "global_title".

#4

Mis à jour par Thomas Noël il y a presque 5 ans

On aura un plantage en cas de gestion des bounces activée, sur cette ligne :

            server_name = msg['From'].split('@')[1]

Or msg['From'] n'est plus une simple adresse email, ça peut être un "HELLO" <foo@bar.com> et dans ce cas, badadoum.

Je serais pour faire :

            try:
                server_name = email_from.split('@')[1]
            except KeyError:
                server_name = 'localhost'

juste avant le calcul de sitename (ligne ~ 260) afin de ne pas avoir à le calculer lors de la gestion des «bounce_handler»

#5

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

On est sûrs™ d'avoir une adresse email dans email_from (et je ne sais pas trop ce que tu attraperais via KeyError). (et je vais faire un ticket pour dégager la gestion des bounces).

#6

Mis à jour par Thomas Noël il y a presque 5 ans

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

Frédéric Péters a écrit :

On est sûrs™ d'avoir une adresse email dans email_from (et je ne sais pas trop ce que tu attraperais via KeyError).

La fatigue, confusion entre msg[From] et email_from, bref, ack.

(et je vais faire un ticket pour dégager la gestion des bounces).

Ouaip... 2019, le mail est mort...

#7

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit 94c00fbefa34aabe02547329e0cd076c3ca33903
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon May 27 13:08:15 2019 +0200

    emails: use global title in From header (#33437)
#8

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

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF