Projet

Général

Profil

Development #62354

erreur sur envoi "compte inutilisé" à une adresse invalide (IndexError: string index out of range)

Ajouté par Sentry Io il y a environ 2 ans. Mis à jour il y a presque 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
03 mars 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

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

  File "authentic2/management/commands/clean-unused-accounts.py", line 131, in send_mail
    send_templated_mail(email, prefix, ctx)
  File "authentic2/utils/misc.py", line 720, in send_templated_mail
    **kwargs,
  File "django/core/mail/__init__.py", line 60, in send_mail
    return mail.send()
  File "django/core/mail/message.py", line 306, in send
    return self.get_connection(fail_silently).send_messages([self])
  File "django/core/mail/backends/smtp.py", line 110, in send_messages
    sent = self._send(message)
  File "hobo/emails/backend.py", line 40, in _send
    return super()._send(email_message)
  File "django/core/mail/backends/smtp.py", line 123, in _send
    recipients = [sanitize_address(addr, encoding) for addr in email_message.recipients()]
  File "django/core/mail/backends/smtp.py", line 123, in <listcomp>
    recipients = [sanitize_address(addr, encoding) for addr in email_message.recipients()]
  File "django/core/mail/message.py", line 134, in sanitize_address
    address = Address(addr_spec=addr)
  File "email/headerregistry.py", line 42, in __init__
    a_s, rest = parser.get_addr_spec(addr_spec)
  File "email/_header_value_parser.py", line 1584, in get_addr_spec
    token, value = get_local_part(value)
  File "email/_header_value_parser.py", line 1394, in get_local_part
    if value[0] in CFWS_LEADER:
...

Fichiers


Demandes liées

Lié à Authentic 2 - Development #67101: build nocturne casséFermé07 juillet 2022

Actions
Dupliqué par Authentic 2 - Bug #66901: clean-unused-accounts sur un compte avec email invalide, IndexError: string index out of rangeFermé03 juillet 2022

Actions

Révisions associées

Révision ca1583f2 (diff)
Ajouté par Benjamin Dauvergne il y a presque 2 ans

utils: do not try to send mail to clearly unusable email addresses (#62354)

Révision d9712705 (diff)
Ajouté par Benjamin Dauvergne il y a presque 2 ans

tests: refactor EmailValidator tests (#62354)

Révision 78e694f8 (diff)
Ajouté par Benjamin Dauvergne il y a presque 2 ans

tests: add test on @ bad email (#62354)

Historique

#1

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

  • Projet changé de Suivi des traces à Authentic 2

D'une lecture rapide ça ferait aussi que la suite des comptes ne seraient pas traités.

#2

Mis à jour par Paul Marillonnet il y a environ 2 ans

Aussi, comprendre comment, avec les systèmes de validation d’adresse qu’on a, quelqu’un a pu s’enregistrer à l’adresse 'prenom@nom@wanadoo.fr'

#3

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

Ça c'est facile il y a de vieux comptes importés sans validation.

#4

Mis à jour par Paul Marillonnet il y a environ 2 ans

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

D'une lecture rapide ça ferait aussi que la suite des comptes ne seraient pas traités.

Oui, aussi.

Ça c'est facile il y a de vieux comptes importés sans validation.

J’ai tapé 'prenom.nom@wanadoo.fr' à la place de l’adresse qui posait souci, mais on peut sans doute aussi carrément supprimer l’usager.

#5

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

  • Dupliqué par Bug #66901: clean-unused-accounts sur un compte avec email invalide, IndexError: string index out of range ajouté
#6

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

  • Assigné à mis à Benjamin Dauvergne
#7

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

Voilà, il faudra peut-être aussi ouvrir des tickets pour ne pas accepter des adresses email invalid des IdPs distants (OIDC, FC, SAML) ou des annuaires (LDAP).

#9

Mis à jour par Corentin Séchet il y a presque 2 ans

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

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit 78e694f81ff84f720d48e22d16cc0232b2aa98a3
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Tue Jul 5 15:57:46 2022 +0200

    tests: add test on @ bad email (#62354)

commit d97127051dd5024ffa9fbaeeaca9ff5db0974b33
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Tue Jul 5 15:56:43 2022 +0200

    tests: refactor EmailValidator tests (#62354)

commit ca1583f28092bcebcc3c6aede6fb41fa0b6cffd9
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Mon Jul 4 10:12:35 2022 +0200

    utils: do not try to send mail to clearly unusable email addresses (#62354)
#11

Mis à jour par A. Berriot il y a presque 2 ans

#12

Mis à jour par Transition automatique il y a presque 2 ans

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

Mis à jour par Transition automatique il y a plus d'un an

Automatic expiration

Formats disponibles : Atom PDF