Development #20454
Retrait de la possibilité de création de rôles A2 depuis les groupes LDAP lors du provisioning
0%
Description
Pas de définition correcte de la façon dont doivent être créés les rôles dans A2 s'il n'existe pas déjà avant le provisioning LDAP.
Donc, pour que le provisioning des rôles se passent proprement (#16523), il faut que ceux-ci existent déjà côté A2.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Paul Marillonnet il y a plus de 6 ans
- Lié à Development #16523: backend LDAP : approvisionnement des rôles depuis les attributs LDAP memberOf ajouté
Mis à jour par Paul Marillonnet il y a plus de 6 ans
- Fichier 0001-WIP-ldap_backend-remove-imprecise-role-creation-capa.patch 0001-WIP-ldap_backend-remove-imprecise-role-creation-capa.patch ajouté
- Patch proposed changé de Non à Oui
Premier patch WIP pour donner une idée.
Je dois encore adapter les tests, à mon avis en mockant la classe des rôles A2.
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Tu peux garder le test nocreate en virant le 'create_role': False, si le rôle n'existe pas on aura l'effet voulu, ça permet de valider qu'on a bien viré la fonctionnalité.
Mis à jour par Paul Marillonnet il y a plus de 6 ans
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Ok mais là on a plus aucun test sur la fonctionnalité set_mandatory_roles, faut juste modifier le premier test plutôt que de le supprimer complètement pour créer au moins un des rôles demandés et vérifier qu'il est bien mis sur l'utilisateur, et renommer le test en 'test_set_mandatory_roles'.
Mis à jour par Paul Marillonnet il y a plus de 6 ans
- Fichier 0001-ldap_backend-remove-imprecise-role-creation-capabili.patch 0001-ldap_backend-remove-imprecise-role-creation-capabili.patch ajouté
Comme ça ?
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Ack mais ne les renomme pas _pytest_truc je ne vois pas trop le besoin.
Mis à jour par Paul Marillonnet il y a plus de 6 ans
- Statut changé de Nouveau à Résolu (à déployer)
Ok j'ai retiré le renommage des rôles.
commit 13390a0af3d00c47f82a3d6c1899deef3a12be2e
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date: Wed Dec 6 10:24:25 2017 +0100
ldap_backend: remove imprecise role creation capability (#20454)
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Fermé
ldap_backend: remove imprecise role creation capability (#20454)