Projet

Général

Profil

Development #42880

Email et casse

Ajouté par Benjamin Dauvergne il y a presque 4 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
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

Lié à Authentic 2 - Development #56392: auth_oidc: ne pas tenir compte de la casse pour la réconciliation par adresse emailFermé26 août 2021

Actions

Révisions associées

Révision e7ed61ff (diff)
Ajouté par Benjamin Dauvergne il y a plus de 2 ans

misc: ignore case when searching by email (#42880)

Historique

#2

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é
#3

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

  • Assigné à mis à Benjamin Dauvergne
#4

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

J'ai fait le tour de tous les .filter(email=...) ou équivalent.

#5

Mis à jour par Valentin Deniaud il y a plus de 2 ans

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

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)
#7

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

Formats disponibles : Atom PDF