Bug #27211
openCannot remove all rights from a user who has a single role, from the User management GUI
Description
When a user is initially created with the GUI, it is created with “no rights”.
As soon as the user has been given one role, it is not possible to remove the only role the user has, because :
- The system refuses to delete the only role attributed to the user ;
- If one tries to add “no rights” again, it is removed when pressing [Save] as the user still has some other rights.
- It is impossible to remove the only rights the user have after having added “no rights” without having saved.
So it is possible to add a different role to a user, then remove the previous one, but it is not possible to remove all rights.
All that can be done is disable or remove the user.
Files
Updated by François ARMAND 17 days ago
- Assignee set to Clark ANDRIANASOLO
- Priority changed from To review to 2
Perhaps something changed in the validation of empty things or UI. But since we are still able to disable users, that doesn't seems very impactful.
Updated by Clark ANDRIANASOLO 10 days ago
- Target version set to 8.2.9
- Severity set to Minor - inconvenience | misleading | easy workaround
- UX impact set to I hate Rudder for that
- User visibility set to First impressions of Rudder
- Effort required set to Very Small
- Priority changed from 0 to 157
Updated by Clark ANDRIANASOLO 10 days ago
- Status changed from New to In progress
Updated by Clark ANDRIANASOLO 10 days ago
The API has something wrong about its behavior, in the representation of "emptiness" of permissions to update VS the intent to "reset" permissions :
it seems like passing "permissions":[]
has the behavior of passing "permissions":null
which does not update permissions at all.
A "permissions":[]
in the update payload should mean that the user has "no permissions", which should end up in the update of the users' permissions.
Then, in the current UI, when the list is empty a right no_right
is displayed, but this should have no x
because it cannot be removed
Updated by Clark ANDRIANASOLO 10 days ago
- Status changed from In progress to Pending technical review
- Pull Request set to https://github.com/Normation/rudder/pull/6525
Updated by Clark ANDRIANASOLO 9 days ago
- Status changed from Pending technical review to Pending release
Applied in changeset rudder|bb5d87fc6b1394bca610bede6be03f5e008dae17.