Projet

Général

Profil

Bug #57510

string index out of range sur adresse email invalide

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
01 octobre 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Exception:
  type = '<class 'IndexError'>', value = 'string index out of range'

Stack trace (most recent call first):
  File "/usr/lib/python3.7/email/_header_value_parser.py", line 1394, in get_local_part
  1392     local_part = LocalPart()
  1393     leader = None
> 1394     if value[0] in CFWS_LEADER:
  1395         leader, value = get_cfws(value)
  1396     if not value:

  locals:
     leader = None
     local_part = LocalPart([])
     value = ''
...
  File "/usr/lib/python3/dist-packages/django/core/mail/backends/smtp.py", line 123, in <listcomp>
   121         encoding = email_message.encoding or settings.DEFAULT_CHARSET
   122         from_email = sanitize_address(email_message.from_email, encoding)
>  123         recipients = [sanitize_address(addr, encoding) for addr in email_message.recipients()]
   124         message = email_message.message()
   125         try:

  locals:
     .0 = <list_iterator object at 0x7f88c51da908>
     addr = 'jeremy.hecre@grandnancy.eu-thierry.laguerre@grandnancy.eu'
     encoding = 'utf-8'

On devrait sans doute en amont vérifier les adresses et faire un record_error en cas de soucis (et ne pas lever d'exception).


Fichiers

Révisions associées

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

misc: only pass sane addresses to email backend (#57510)

Historique

#1

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

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

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

(option d'ignorer les éventuelles mauvaises adresses, sans les logguer).

#3

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

(mauvais patch attaché)

#5

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

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

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 5f08140cc8564a7bf4bc2e0fe1ab0d0eb46c3355
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Thu Nov 11 18:52:50 2021 +0100

    misc: only pass sane addresses to email backend (#57510)
#7

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