Project

General

Profile

Développement #99138

Rendre insensible à la casse l'assignation d'un utilisateur à une fonction sur la base de l'adresse mail

Added by Emmanuel Cazenave 4 days ago. Updated about 2 hours ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
28 November 2024
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

Ce n'est actuellement pas le cas, l'assignation échoue s'il y a une différence de casse.


Files

patch.txt (498 Bytes) patch.txt Emmanuel Cazenave, 28 November 2024 12:13 PM

Associated revisions

Revision b839a23a (diff)
Added by Frédéric Péters about 11 hours ago

misc: ignore email case when looking up users (#99138)

History

#3

Updated by Benjamin Dauvergne 4 days ago

Il faut rajouter un index 'users_email_upper_idx ON users (lower(email))',[1]ensuite tu peux utiliser IEqual(...) sans ralentissement.

Coté authentic on a une passe de revérification des emails après la recherche en base, https://git.entrouvert.org/entrouvert/authentic/src/branch/main/src/authentic2/custom_user/managers.py#L196 ça peut valoir le coup de reprendre ça aussi.

1 En Django la même chose utilise toujours UPPER, jamais compris s'il y avait un intérêt à l'un ou l'autre.

#4

Updated by Robot Gitea about 12 hours ago

  • Status changed from Nouveau to En cours
  • Assignee set to Frédéric Péters

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

#5

Updated by Robot Gitea about 11 hours ago

  • Status changed from En cours to Solution proposée
#6

Updated by Robot Gitea about 11 hours ago

  • Status changed from Solution proposée to Solution validée

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

#7

Updated by Robot Gitea about 11 hours ago

  • Status changed from Solution validée to Résolu (à déployer)

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

#8

Updated by Transition automatique about 2 hours ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF