Projet

Général

Profil

Bug #49623

"@" comme adresse email (→ IndexError: string index out of range)

Ajouté par Sentry Io il y a plus de 3 ans. Mis à jour il y a 5 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
19 décembre 2020
Echéance:
% réalisé:

0%

Temps estimé:
Hors marché:
Non
Patch proposed:
Non
Planning:
Non

Description

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

Un compte qui a "@" comme adresse, marquée vérifiée même (cf capture), crash au moment de l'envoi d'un email :

IndexError: string index out of range
(15 additional frame(s) were not displayed)
...
  File "django/core/mail/backends/smtp.py", line 124, in <listcomp>
    recipients = [sanitize_address(addr, encoding) for addr in email_message.recipients()]
  File "django/core/mail/message.py", line 161, in sanitize_address
    address = Address(nm, 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:

J'imagine que c'est au retour de fédération FC que l'email est modifié et que ce qui a été fourni comme adresse est totalement mauvais; qu'il y aurait à vérifier que l'adresse reçue est bien formatée.


Fichiers

Screenshot_2020-12-19 Connexion.png (2,89 ko) Screenshot_2020-12-19 Connexion.png Frédéric Péters, 19 décembre 2020 22:42

Révisions associées

Révision e0dcfc9c (diff)
Ajouté par Benjamin Dauvergne il y a 5 mois

auth_fc: ignore invalid emails (#49623)

Historique

#1

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

#5

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

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

Cette fois un compte avec une adresse de courriel contenant deux arobases.

#6

Mis à jour par Benjamin Dauvergne il y a 5 mois

  • Assigné à mis à Benjamin Dauvergne
#7

Mis à jour par Robot Gitea il y a 5 mois

  • Statut changé de Nouveau à Solution proposée

Benjamin Dauvergne (bdauvergne) a ouvert une pull request sur Gitea concernant cette demande :

#8

Mis à jour par Robot Gitea il y a 5 mois

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

Serghei Mihai (smihai) a approuvé une pull request sur Gitea concernant cette demande :

#9

Mis à jour par Robot Gitea il y a 5 mois

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

Benjamin Dauvergne (bdauvergne) a mergé une pull request sur Gitea concernant cette demande :

#10

Mis à jour par Transition automatique il y a 5 mois

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

Mis à jour par Transition automatique il y a 3 mois

Automatic expiration

Formats disponibles : Atom PDF