Projet

Général

Profil

Bug #46484

authentic2_auth_saml: en cas de création de l'usager, le mettre dans la collectivité par défaut.

Ajouté par Thomas Noël il y a plus de 3 ans. Mis à jour il y a plus de 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Catégorie:
-
Version cible:
-
Début:
09 septembre 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Ca ne semble pas être le cas et je n'ai pas vu dans le code à quel moment ça pourrait effectivement intervenir.


Fichiers

Révisions associées

Révision 9f08f5c4 (diff)
Ajouté par Frédéric Péters il y a plus de 3 ans

auth saml: put newly created user in default OU (#46484)

Historique

#2

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

Ils doivent être positionnées dans une collectivité, et par défaut celle par défaut oui.

#3

Mis à jour par Thomas Noël il y a plus de 3 ans

Benjamin Dauvergne a écrit :

Ils doivent être positionnées dans une collectivité, et par défaut celle par défaut oui.

Alors y'a quelque chose qui m'échappe dans #46473 et #46389

#4

Mis à jour par Thomas Noël il y a plus de 3 ans

Dans le code de création de l'utilisateur de authentic2_auth_saml/adapters.py je ne vois que :

    def create_user(self, user_class):
        return user_class.objects.create()

    def finish_create_user(self, idp, saml_attributes, user):
        try:
            self.provision_a2_attributes(user, idp, saml_attributes)
        except MappingError as e:
            raise UserCreationError('user creation failed on a mandatory mapping action: %s' % e)

    def provision(self, user, idp, saml_attributes):
        super(AuthenticAdapter, self).provision(user, idp, saml_attributes)
        try:
            self.provision_a2_attributes(user, idp, saml_attributes)
        except MappingError as e:
            logger.warning('auth_saml: failure during attribute provisionning %s', e)

...

et donc rien qui ne donne l'ou à l'utilisateur.

Dans la partie OIDC on a bien « user = User.objects.create(ou=provider.ou) » dont je ne trouve pas l'équivalent.

#5

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

  • Sujet changé de mellon: en cas de création de l'usager, le mettre dans la collectivité par défaut. à authentic2_auth_saml: en cas de création de l'usager, le mettre dans la collectivité par défaut.

Je ne comprends pas à quoi tu réponds, je suis d'accord.

PS: je dis juste que si on peut aussi poser les gens dans une autre OU quand on en a besoin, c'est bien.

#6

Mis à jour par Frédéric Péters il y a plus de 3 ans

Thomas a lu "Ils doivent être positionnées" comme étant la situation présente, et pas une confirmation comme quoi ce n'était pas le cas mais une bonne chose à faire.

#7

Mis à jour par Frédéric Péters il y a plus de 3 ans

#8

Mis à jour par Thomas Noël il y a plus de 3 ans

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

So simple, j'ai un peu honte.

#9

Mis à jour par Frédéric Péters il y a plus de 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 9f08f5c47513144910636f3aec7c0fa45049a7b6
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Oct 27 18:26:55 2020 +0100

    auth saml: put newly created user in default OU (#46484)
#10

Mis à jour par Frédéric Péters il y a plus de 3 ans

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

Formats disponibles : Atom PDF