Projet

Général

Profil

Development #48679

ne pas laisser passer les accents dans la partie gauche d'une adresse email (user)

Ajouté par Thomas Noël il y a plus de 3 ans. Mis à jour il y a plus de 3 ans.

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

On voit des personnes saisir une adresse email avec des accents, par exemple thomas.noë

C'est autorisé par les RFC... ou presque, cf https://tools.ietf.org/html/rfc6532#section-3.2 qui est juste "PROPOSED" et selon https://stackoverflow.com/questions/2049502/what-characters-are-allowed-in-an-email-address "as of 2019 these standards are still marked as Proposed, but are being rolled out slowly."

Bref, aujourd'hui il s'agit la plupart du temps d'erreurs de saisie.

Je proposerais que w.c.s. valide la partie gauche avec la même regex que Django (et qu'on voit ailleurs) :

# extrait de https://github.com/django/django/blob/master/django/core/validators.py
    user_regex = _lazy_re_compile(
        r"(^[-!#$%&'*+/=?^_`{}|~0-9A-Z]+(\.[-!#$%&'*+/=?^_`{}|~0-9A-Z]+)*\Z"  # dot-atom
        r'|^"([\001-\010\013\014\016-\037!#-\[\]-\177]|\\[\001-\011\013\014\016-\177])*"\Z)',  # quoted-string
        re.IGNORECASE)


Fichiers

Révisions associées

Révision 44b27d21 (diff)
Ajouté par Thomas Noël il y a plus de 3 ans

emails: refuse non standard user part in email addresses validation (#48679)

Historique

#1

Mis à jour par Thomas Noël il y a plus de 3 ans

#2

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

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

Tu peux ajouter un pointeur vers le fichiers des sources de django d'où ça sort ?

Et je préférais mes elif à ces return, et sans doute que la regex doit permettre de dégager une série de ces tests, mais laissons ce nettoyage à plus tard.

#3

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

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

Et j'ai donc poussé ainsi pour pouvoir tagguer.

commit 44b27d21646b5a6b9ea65f51c94555c968593440
Author: Thomas Noël <tnoel@entrouvert.com>
Date:   Fri Nov 20 01:32:23 2020 +0100

    emails: refuse non standard user part in email addresses validation (#48679)
#4

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

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

Formats disponibles : Atom PDF