Projet

Général

Profil

Bug #63730

auth_oidc : faux message « created user ... »

Ajouté par Thomas Noël il y a environ 2 ans. Mis à jour il y a environ 2 ans.

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

Révision ec5cc3e3 (diff)
Ajouté par Thomas Noël il y a environ 2 ans

auth_oidc: log created user only if a user is created (#63730)

Historique

#1

Mis à jour par Thomas Noël il y a environ 2 ans

#2

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.

#3

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)

#4

Mis à jour par Transition automatique il y a environ 2 ans

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

Mis à jour par Transition automatique il y a presque 2 ans

Automatic expiration

Formats disponibles : Atom PDF