- Issue created by @plopesc
- Status changed to Active
about 1 year ago 3:56pm 5 March 2024 - 🇪🇸Spain plopesc Valladolid
Given that progress in the issue this one is postponed on 📌 Add a container parameter that can remove the special behavior of UID#1 Fixed is not going as fast as we would like to, we decided to work on this one to unblock it.
- Merge request !7Issue #3395483 by plopesc, penyaskito: Define dashboard behavior for user 1 → (Merged) created by plopesc
- Status changed to Needs review
about 1 year ago 8:53am 6 March 2024 - 🇪🇸Spain plopesc Valladolid
After some thinking, created a MR where the logic has been modified to check for the specific permission instead of checking if the permission belongs to any of the user roles.
The result of this change is hat user 1 will have access to all the published dashboards, while the res of users logic will remain as it was.
I think this is a compromise solution until the arrival of 📌 Add a container parameter that can remove the special behavior of UID#1 Fixed .
- 🇪🇸Spain penyaskito Seville 💃, Spain 🇪🇸, UTC+2 🇪🇺
Given how close 📌 Add a container parameter that can remove the special behavior of UID#1 Fixed might be to land, this looks like the right call.
- 🇪🇸Spain penyaskito Seville 💃, Spain 🇪🇸, UTC+2 🇪🇺
Oh, so close that it landed a week ago! Still current behavior remains by default though.
- 🇪🇸Spain plopesc Valladolid
Yep, I believe we will still to use this workaround - or a different one - for now.
- 🇪🇸Spain penyaskito Seville 💃, Spain 🇪🇸, UTC+2 🇪🇺
I like the MR, is this something we can add a test for?
- 🇪🇸Spain plopesc Valladolid
Good point!
I don't know well how to replicate the user 1 behavior in kernel tests after 📌 Add a container parameter that can remove the special behavior of UID#1 Fixed .
Actually this MR is agnostic and gives the actual responsibility to the site administrator.
If they decide to give superpowers to user 1, they'll access to all the dashboards.
If user 1 is not empowered, it would behave like any other.
Don't think we need to add extra tests to confirm the default behavior.
If you feel safer having them, we could explore how to make it possible.
- Status changed to RTBC
about 1 year ago 3:02pm 16 April 2024 - Status changed to Fixed
about 1 year ago 3:04pm 16 April 2024 - 🇪🇸Spain plopesc Valladolid
MR merged. Opened 📌 [PP] Define specific tests for disabled super user access policy Active to implement extra tests for disabled super user access policy once it is added to a stable release.
Automatically closed - issue fixed for 2 weeks with no activity.