Projet

Général

Profil

Bug #80385

forcer un strip() des emails (BadHeaderError: Header values can't contain newlines (got 'xxx@orange.fr\n' for header 'To'))

Ajouté par Sentry Io il y a 9 mois. Mis à jour il y a 9 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
12 août 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Sans chercher l'origine du \n on pourrait juste strip().

https://sentry.entrouvert.org/entrouvert/publik/issues/113173/

BadHeaderError: Header values can't contain newlines (got 'xxx@orange.fr\n' for header 'To')
(5 additional frame(s) were not displayed)
...
  File "django/core/mail/backends/smtp.py", line 123, in _send
    message = email_message.message()
  File "django/core/mail/message.py", line 250, in message
    self._set_list_header_if_not_empty(msg, 'To', self.to)
  File "django/core/mail/message.py", line 405, in _set_list_header_if_not_empty
    msg[header] = value
  File "django/core/mail/message.py", line 184, in __setitem__
    name, val = forbid_multi_line_headers(name, val, self.encoding)
  File "django/core/mail/message.py", line 60, in forbid_multi_line_headers
    raise BadHeaderError("Header values can't contain newlines (got %r for header %r)" % (val, name))

Révisions associées

Révision bb13724b (diff)
Ajouté par Frédéric Péters il y a 9 mois

misc: force stripping of email addresses (#80385)

Historique

#1

Mis à jour par Frédéric Péters il y a 9 mois

  • Projet changé de Suivi des traces à w.c.s.
#2

Mis à jour par Robot Gitea il y a 9 mois

  • Statut changé de Nouveau à Solution proposée
  • Assigné à mis à Frédéric Péters

Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :

#3

Mis à jour par Robot Gitea il y a 9 mois

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

Valentin Deniaud (vdeniaud) a approuvé une pull request sur Gitea concernant cette demande :

#4

Mis à jour par Robot Gitea il y a 9 mois

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

Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :

#5

Mis à jour par Transition automatique il y a 9 mois

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

Mis à jour par Transition automatique il y a 7 mois

Automatic expiration

Formats disponibles : Atom PDF