Bug #8558
Change to the admin roles permissions are not applied
Début:
08 octobre 2015
Echéance:
% réalisé:
100%
Temps estimé:
Patch proposed:
Non
Planning:
Description
This code in update_ous_admin_roles() prevents update_ou_admin role to be called for all ou:
ou_ids = ou_all.values_list('id', flat=True) ... perm_ids_with_role = Role.objects.filter( admin_scope_ct=ContentType.objects.get_for_model(Permission) ).values_list('admin_scope_id') ou_admin_perms = Permission.objects.filter( operation__slug='view', target_ct=ContentType.objects.get_for_model(OU), id__in=perm_ids_with_role) ou_ids_with_perm = ou_admin_perms.values_list('target_id', flat=True) ... for ou in OU.objects.filter(id__in=set(ou_ids)-set(ou_ids_with_perm)): update_ou_admin_roles(ou)
But it prevents too much.
Demandes liées
Révisions associées
a2_rbac: fix undefined variables errors (fixes #8558)
Also some pep8 enhancements.
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
- Statut changé de Nouveau à Résolu (à déployer)
- % réalisé changé de 0 à 100
Appliqué par commit authentic2|a058c274a6caf9b0acd7e26306ff44296a20cf97.
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Lié à Bug #8565: NameError: global name 'ou_admin_perms' is not defined ajouté
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
Appliqué par commit authentic2|55c5709023f701ff895e38f485e7ca81d5d2fb98.
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Statut changé de Solution déployée à Fermé
a2_rbac/management: remove broken optimization in update_ous_admin_roles (fixes #8558)
Trying to filter ou before updating their admin roles fails because the
filtering is too simple, so we lack some chanes, for example adding the
vire permission on organizational unit to all admin roles.