Projet

Général

Profil

Bug #81014

Erreur sur l'action courriel à cause d'un courriel mal formaté

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
08 septembre 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

https://sentry.entrouvert.org/entrouvert/grenoble/issues/115001/

UnicodeEncodeError: 'ascii' codec can't encode character '\xe9' in position 7: ordinal not in range(128)
  File "django/core/mail/message.py", line 62, in forbid_multi_line_headers
    val.encode('ascii')

ValueError: not enough values to unpack (expected 2, got 1)
(15 additional frame(s) were not displayed)
...
  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 65, in forbid_multi_line_headers
    val = ', '.join(sanitize_address(addr, encoding) for addr in getaddresses((val,)))
  File "django/core/mail/message.py", line 65, in <genexpr>
    val = ', '.join(sanitize_address(addr, encoding) for addr in getaddresses((val,)))
  File "django/core/mail/message.py", line 97, in sanitize_address
    localpart, domain = address.rsplit('@', 1)

Révisions associées

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

emails: do not accept as sane addresses without @ (#81014)

Historique

#1

Mis à jour par Nicolas Roche il y a 8 mois

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

Erreur sur l'action courriel
https://services.demarches.grenoblealpesmetropole.fr/backoffice/workflows/120/status/4/items/1/
sur un formulaire où l'usager à rempli le champ courriel de type texte avec son nom
https://services.demarches.grenoblealpesmetropole.fr/backoffice/management/metro-rando/2964/

Il y a une première exception parce que le nom contient des accents,
puis seconde exception est remontée dans sentry parce qu'il n'y a pas d'arobase dans le courriel fourni.

J'ai fait un ticket client pour qu'il utilise un champ de type courriel,
#81013
mais je me demande si l'on ne pourrait pas alerter d'une façon ou d'une autre, lorsqu'on défini un champ nommé courriel de type autre que courriel.

#3

Mis à jour par Robot Gitea il y a 8 mois

  • Statut changé de Nouveau à En cours
  • Assigné à mis à Frédéric Péters

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

#4

Mis à jour par Robot Gitea il y a 8 mois

  • Statut changé de En cours à Solution proposée
#5

Mis à jour par Robot Gitea il y a 8 mois

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

Lauréline Guérin (lguerin) a approuvé une pull request sur Gitea concernant cette demande :

#6

Mis à jour par Robot Gitea il y a 8 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 :

#7

Mis à jour par Transition automatique il y a 8 mois

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

Mis à jour par Transition automatique il y a 6 mois

Automatic expiration

Formats disponibles : Atom PDF