Development #71267
a2_rbac : avoir un rôle interne d’administration des clients d’API
0%
Description
Actuellement on se repose sur le rôle interne créé pour l’administration des services (/manage/services/), d’une part ça peut porter à confusion et d’autre part c’est à mon avis souhaitable que ce soit deux choses séparées au niveau du contrôle d’accès.
Files
Related issues
Associated revisions
History
Updated by Paul Marillonnet 3 months ago
- Related to Development #71275: contrôle d’accès : gérer l’appartenance à des collectivités pour les clients d’API added
Updated by Paul Marillonnet 3 months ago
- File 0001-a2_rbac-add-global-management-role-for-api-clients-7.patch 0001-a2_rbac-add-global-management-role-for-api-clients-7.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Le patche juste pour le rôle global. Pour des rôles par OU (comme on fait pour d’autres objets d’administration dans a2), il faudra #71275 d’abord.
Updated by Valentin Deniaud 3 months ago
Updated by Paul Marillonnet 3 months ago
- Related to Development #71284: tests : prendre en compte le nouveau rôle interne d’administration des clients d’API added
Updated by Paul Marillonnet 3 months ago
- File 0001-a2_rbac-add-global-management-role-for-api-clients-7.patch 0001-a2_rbac-add-global-management-role-for-api-clients-7.patch added
Valentin Deniaud a écrit :
Quand j'ai fait la même chose côté moyens d'authentification dans #66984 j'ai mis à jour les commentaires des tests également, si jamais tu as la déter :)
Bien vu, j’avais loupé ceux au début des test_a2_rbac en effet.
Aussi il y a un commit à faire côté hobo (#70866)
J’ai créé #71284, je regarde.
et la migration sera lente en recette (#70958, peut-être un script à faire tourner avant).
Si c’est bon pour toi je verrai le temps de mise à jour des recettes et ferai si nécessaire un script, comme celui du ticket mentionné, pour les productions.
Updated by Valentin Deniaud 3 months ago
Paul Marillonnet a écrit :
Valentin Deniaud a écrit :
et la migration sera lente en recette (#70958, peut-être un script à faire tourner avant).
Si c’est bon pour toi je verrai le temps de mise à jour des recettes et ferai si nécessaire un script, comme celui du ticket mentionné, pour les productions.
Je voulais bien dire faire tourner le script en recette, sinon c'est certain qu'il y aura 30 minutes et quelques de traces (et un ticket de Fred pour ta pomme !).
Et d'après le même protocole expérimental, ça n'est pas utile de le faire tourner en prod (mais eût trait à la rigueur relative du protocole, ça ne mange pas de pain).
Updated by Paul Marillonnet 3 months ago
- File 0001-commands-add-api-client-admin-role-creation-command-.patch 0001-commands-add-api-client-admin-role-creation-command-.patch added
Valentin Deniaud a écrit :
Paul Marillonnet a écrit :
Valentin Deniaud a écrit :
et la migration sera lente en recette (#70958, peut-être un script à faire tourner avant).
Je voulais bien dire faire tourner le script en recette, sinon c'est certain qu'il y aura 30 minutes et quelques de traces (et un ticket de Fred pour ta pomme !).
Et d'après le même protocole expérimental, ça n'est pas utile de le faire tourner en prod (mais eût trait à la rigueur relative du protocole, ça ne mange pas de pain).
Ok, je peux faire tourner ça, en recette avant le déploiement, et en prod avant le 24 au soir.
Le commit est poussé dans la branche pour que Jenkins valide le test, mais je ne pousserai pas ça dans main.
Updated by Valentin Deniaud 3 months ago
- Status changed from Solution proposée to Solution validée
Vu qu'il n'y a pas de champ collectivité au modèle APIClient je me dis que la ligne scoped_name
n'est pas utile, mais ça ne me paraît pas très important.
Updated by Paul Marillonnet 2 months ago
- Status changed from Solution validée to Résolu (à déployer)
commit 5a821a88838d8878b9dd46458bccbcaf2fd9017f Author: Paul Marillonnet <pmarillonnet@entrouvert.com> Date: Mon Nov 14 12:26:17 2022 +0100 a2_rbac: add global management role for api clients (#71267) ou-wise api-client management roles will be added in #71275.
Updated by Transition automatique about 1 month ago
- Status changed from Résolu (à déployer) to Solution déployée
a2_rbac: add global management role for api clients (#71267)