Projet

Général

Profil

Bug #57354

traces vs backend email django (BadHeaderError: Header values can't contain newlines)

Ajouté par Sentry Io il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Frédéric Péters
Version cible:
-
Début:
28 septembre 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

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

BadHeaderError: Header values can't contain newlines (got 'Montpellier\xa0: [ERROR] [CRON] psycopg2.ProgrammingError: column "f5" does not exist\nLINE 1: ...f9, f12, f12_display, f12_structured, f2, f3, f4, f5, f1, f1...\n                                                             ^' for header 'Subject')
(13 additional frame(s) were not displayed)
...
  File "hobo/emails/backend.py", line 40, in _send
    return super()._send(email_message)
  File "django/core/mail/backends/smtp.py", line 124, in _send
    message = email_message.message()
  File "django/core/mail/message.py", line 270, in message
    msg['Subject'] = self.subject
  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 62, in forbid_multi_line_headers
    raise BadHeaderError("Header values can't contain newlines (got %r for header %r)" % (val, name))

Fichiers

0001-errors-limit-length-and-remove-newline-characters-fr.patch (907 octets) 0001-errors-limit-length-and-remove-newline-characters-fr.patch Frédéric Péters, 28 septembre 2021 15:55

Révisions associées

Révision 56723543 (diff)
Ajouté par Frédéric Péters il y a plus de 2 ans

errors: limit length and remove newline characters from summaries (#57354)

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 2 ans

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

Soit remplacer les retours à la ligne par des espaces soit couper la chaine au premier retour à la ligne, je dirais.

#2

Mis à jour par Frédéric Péters il y a plus de 2 ans

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

Mis à jour par Frédéric Péters il y a plus de 2 ans

En passant je coupe aussi à 400 caractères de souvenirs de traces passées ou l'SMTP n'était pas content avec des sujets mégalongs. (mais je tire la valeur au hasard).

#4

Mis à jour par Lauréline Guérin il y a plus de 2 ans

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

Mis à jour par Frédéric Péters il y a plus de 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 5672354372bbb23c1dce86a8d0c61c8a1a274de1
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Sep 28 15:54:33 2021 +0200

    errors: limit length and remove newline characters from summaries (#57354)
#6

Mis à jour par Frédéric Péters il y a plus de 2 ans

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

Formats disponibles : Atom PDF