Bug #46484
authentic2_auth_saml: en cas de création de l'usager, le mettre dans la collectivité par défaut.
0%
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
Historique
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.
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
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.
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.
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.
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Fichier 0001-auth-saml-put-newly-created-user-in-default-OU-46484.patch 0001-auth-saml-put-newly-created-user-in-default-OU-46484.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
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.
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)
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
auth saml: put newly created user in default OU (#46484)