Project

General

Profile

Development #71855

Restaurer du code supprimé abusivement en même temps que le modèle RoleAttribute

Added by Valentin Deniaud 2 months ago. Updated about 2 months ago.

Status:
Solution déployée
Priority:
Normal
Category:
-
Target version:
-
Start date:
29 November 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Actuellement les rôles superutilisateurs des briques sont mal provisionnés en recette à cause de #69895#note-3.


Files

Associated revisions

Revision 8e6a95b6 (diff)
Added by Valentin Deniaud 2 months ago

attributes_ng: restore setting superuser flag (#71855)

History

#3

Updated by Valentin Deniaud 2 months ago

Voilà il y a à restaurer le code + l'adapter (puisque je l'avait viré par flemme de l'adapter), ça aurait été plus clair en deux commits mais je suis pressé.

Testé en local que je ne reproduis plus le bug avec le patch.

#4

Updated by Frédéric Péters 2 months ago

Il ne manque pas un truc pour poser aussi un attribut role-slug ?

#5

Updated by Valentin Deniaud 2 months ago

Frédéric Péters a écrit :

Il ne manque pas un truc pour poser aussi un attribut role-slug ?

Non car pour moi cet attribut n'est plus utilisé, en tout cas un potentiel bug lié à son absence n'a pas été remonté.

#6

Updated by Frédéric Péters 2 months ago

qui attention ne contiendrait pas le slug du rôle mais son uuid, et en fait ça arriverait via a2_service_ou_role_uuids qui arrive via src/authentic2/attributes_ng/sources/django_user.py (donc ça serait ok).

#7

Updated by Frédéric Péters 2 months ago

  • Status changed from Solution proposée to Solution validée

arrive via src/authentic2/attributes_ng/sources/django_user.py

Je confirme que ça vient de là et que malgré le nom qui laisse penser que ça serait en rapport avec un rôle c'est bien la liste des uuid des rôles de l'utilisateur.

#8

Updated by Valentin Deniaud 2 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 8e6a95b6ce813af0f5c3569f9ac13c4c5de10324
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Tue Nov 29 18:30:06 2022 +0100

    attributes_ng: restore setting superuser flag (#71855)
#10

Updated by Transition automatique about 2 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF