Development #42880
Email et casse
Début:
14 mai 2020
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Beaucoup d'endroit où on ignore pas la casse quand on recherche un utilisateur par mail :
src/authentic2/api_views.py:304: qs = User.objects.filter(email=data['email']) src/authentic2/api_views.py:489: if app_settings.A2_EMAIL_IS_UNIQUE and qs.filter(email=data['email']).exists(): src/authentic2/manager/forms.py:224: if a2_app_settings.A2_EMAIL_IS_UNIQUE and qs.filter(email=email).exists(): src/authentic2/utils/__init__.py:728: existing_accounts = User.objects.filter(email=email) src/authentic2/utils/__init__.py:1088: email_is_not_unique = qs.filter(email=email).exclude(pk=user.pk).exists() src/authentic2/utils/__init__.py:1090: email_is_not_unique = qs.filter(email=email, ou=user.ou).exclude(pk=user.pk).exists() src/authentic2/views.py:223: non_unique = User.objects.filter(email=email).exclude(pk=user_pk).exists() src/authentic2/views.py:225: non_unique = User.objects.filter(email=email, ou=user.ou).exclude(
Il faudrait les corriger tous en utilisant iexact et aussi vérifier qu'on conserve bien la casse à la création par contre (pour les pauvres malheureux pour qui ça peut compter), au moins sur la partie locale et aussi striper avant stockage (ça concerne l'API, la vue d'enregistrement, le connecteur LDAP et les connecteurs d'authent SAML, FC et OIDC).
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Lié à Development #56392: auth_oidc: ne pas tenir compte de la casse pour la réconciliation par adresse email ajouté
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Statut changé de Nouveau à Solution proposée
- Fichier 0001-misc-ignore-case-when-searching-by-email-42880.patch 0001-misc-ignore-case-when-searching-by-email-42880.patch ajouté
- Tracker changé de Bug à Development
- Patch proposed changé de Non à Oui
J'ai fait le tour de tous les .filter(email=...)
ou équivalent.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit e7ed61ff3a885b8337694267b2da7631a73d54ed Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Fri Aug 27 14:28:32 2021 +0200 misc: ignore case when searching by email (#42880)
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
misc: ignore case when searching by email (#42880)