Development #72703: /manage/ : l’écran de gestion des clients d’API doit filtrer les rôles ajoutables au client en fonction de l’OU du client d’API - Authentic 2 - Redmine Entr'ouvert

Development #72703

/manage/ : l’écran de gestion des clients d’API doit filtrer les rôles ajoutables au client en fonction de l’OU du client d’API

Added by Paul Marillonnet 3 months ago. Updated 28 days ago.

Status:

Solution déployée

Priority:

Normal

Assignee:

Paul Marillonnet

Category:

Target version:

Start date:

22 December 2022

Due date:

% Done:

Estimated time:

Patch proposed:

Yes

Planning:

Description

Une première étape du travail de cohérence des OUs des clients d’API dans le /manage/ est #72688, qui filtre les clients visibles et l’OU ajoutable au client en fonction du/des rôle(s) de gestion détenus par l’usager.

Une autre partie nécessaire, dont il est question ici, est d’utiliser cette même logique pour filtrer les rôles que l’usager peut ajouter au client d’API

Files

0004-OU-consistency-check-between-api-client-and-roles-at.patch (3.96 KB) 0004-OU-consistency-check-between-api-client-and-roles-at.patch		Paul Marillonnet, 23 December 2022 12:51 PM
0003-manager-filter-api-client-s-assignable-roles-dependi.patch (4.37 KB) 0003-manager-filter-api-client-s-assignable-roles-dependi.patch		Paul Marillonnet, 23 December 2022 12:51 PM
0002-manager-force-api-client-ou-assignment-for-local-adm.patch (3.48 KB) 0002-manager-force-api-client-ou-assignment-for-local-adm.patch		Paul Marillonnet, 23 December 2022 12:51 PM
0001-models-drop-non-nullity-constraint-ou-API-clients-OU.patch (1.85 KB) 0001-models-drop-non-nullity-constraint-ou-API-clients-OU.patch		Paul Marillonnet, 23 December 2022 12:51 PM

Related issues

History

Updated by Paul Marillonnet 3 months ago

Related to Development #72688: /manage/ : l’écran de gestion des clients d’API doit filtrer en fonction de l’OU du/des rôles de gestion détenus par l’usager added

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

Assignee set to Paul Marillonnet

Concrètement, les rôles liés à au moins une permission globale ou dans une OU autre que celle(s) du/des rôle(s) de gestion détenus par l’usager ne devraient pas apparaître parmi la liste des rôles ajoutables au client d’API

Updated by Paul Marillonnet 3 months ago

Subject changed from /manage/ : l’écran de gestion des clients d’API doit filtrer les rôles ajoutables au client en fonction de l’OU du/des rôles de gestion détenus par l’usager to /manage/ : l’écran de gestion des clients d’API doit filtrer les rôles ajoutables au client en fonction de l’OU du client d’API

Faisons plus simple : seuls les rôles pour lesquels role.ou == api_client.ou apparaissent dans la liste des rôles ajoutables au client.

Updated by Paul Marillonnet 3 months ago

File 0004-OU-consistency-check-between-api-client-and-roles-at.patch 0004-OU-consistency-check-between-api-client-and-roles-at.patch added
File 0003-manager-filter-api-client-s-assignable-roles-dependi.patch 0003-manager-filter-api-client-s-assignable-roles-dependi.patch added
File 0002-manager-force-api-client-ou-assignment-for-local-adm.patch 0002-manager-force-api-client-ou-assignment-for-local-adm.patch added
File 0001-models-drop-non-nullity-constraint-ou-API-clients-OU.patch 0001-models-drop-non-nullity-constraint-ou-API-clients-OU.patch added
Status changed from Nouveau to Solution proposée
Patch proposed changed from No to Yes

0001 qui relève la contrainte de non-nullité sur l’OU du client d’API, car dorénavant si on souhaite un client qui tape dans l’API sur toutes les OU ça se matérialise par api_client.ou = None.
0002 qui empêche les admins locaux de créer un tel client d’API "global", en retirant l’option ou = None du formulaire d’ajout/d’édition.
0003 qui, à l’affichage du formulaire, filtre les rôles ajoutables à l’OU en fonction de l’OU d’appartenance du client.
0004 qui, à la validation du formulaire, vérifie la cohérence entre OU du client et OU des rôles ajoutés.

Updated by Paul Marillonnet 3 months ago

(Et oui, j’oubliais, c’est basé sur la branche de #72688.)

Updated by Paul Marillonnet 3 months ago

(À noter aussi qu’on pourrait ajouter à 0004 du JS qui rafraîchit la liste des rôles sélectionnés et celle des rôles disponibles dès que l’OU change dans le formulaire, mais j’ai pas tellement envie de me lancer là dedans. Si quelqu’un veut prendre, welcome. L’essentiel, pour moi, est de conserver la cohérence des OUs, tant pis si l’admin doit s’y reprendre à deux fois pour valider le formulaire.)