Bug #63730
auth_oidc : faux message « created user ... »
Début:
08 avril 2022
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Dans le cas d'une OU avec unicité de mail, et en stratégie STRATEGY_CREATE, si on a déjà un utilisateur avec le mail, on a quand même dans les logs ce message :
auth_oidc: created user ... for sub ... and issuer ...
Je pense qu'il y a un problème dans le code src/authentic2_auth_oidc/backends.py :
... if not user: user = User.objects.create(ou=provider.ou) user.set_unusable_password() created = True oidc_account, created = models.OIDCAccount.objects.get_or_create( <-- ici on utilise "created" mais ce n'est pas pour créer un utilisateur provider=provider, user=user, defaults={'sub': id_token.sub} ) ...
Le created se confond entre la création d'un utilisateur et la création d'un models.OIDCAccount. Ce qui fait que le message "created user" est affiché parce que :
if created: logger.info( 'auth_oidc: created user %s for sub %s and issuer %s', user, id_token.sub, id_token.iss )
mais en fait non, c'est juste un OIDCAccount qui a été créé.
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a environ 2 ans
- Fichier 0001-auth_oidc-log-created-user-only-if-a-user-is-created.patch 0001-auth_oidc-log-created-user-only-if-a-user-is-created.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Serghei Mihai il y a environ 2 ans
- Statut changé de Solution proposée à Solution validée
- Assigné à mis à Thomas Noël
Go quand jenkins vert.
Mis à jour par Thomas Noël il y a environ 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit ec5cc3e34634c97f683c028c1d23339204b9e5cc Author: Thomas NOËL <tnoel@entrouvert.com> Date: Fri Apr 8 15:23:45 2022 +0200 auth_oidc: log created user only if a user is created (#63730)
Mis à jour par Transition automatique il y a environ 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
auth_oidc: log created user only if a user is created (#63730)