Feature #7051
closed
Allow control of what users can view and/or clear notices
Added by Phillip Davis almost 8 years ago.
Updated almost 8 years ago.
Description
Use case:
A user with minimal page privs (e.g. can just change their password, or access a few status pages or...) should not be automatically able to see notices or clear them. Because notices might contain critical system information which discloses some problem with the system, and clearing them would prevent a full firewall administrator from seeing them.
Users with priv for all pages (which includes the built-in admin) should be able to view and clear notices.
Provide 2 new privs that allow:
a) View notices (but cannot clear)
b) View and clear notices
This is a follow-on from bug #3454
- Status changed from New to Feedback
- % Done changed from 0 to 100
PR has been merged, thanks!
- Status changed from Feedback to Assigned
The notice alert/bell isn't displayed to the admin user when this code is in place. If I revert it, they show up.
More info: This appears to have happened because the 'admin' user on that VM was somehow not a member of the 'admins' group. Other access code in the GUI still considers the 'admin' user to have 'admin'-level access in that case.
The code checks for having the specific new privs to view/clear notices or the "all pages" access. If the "root" user called "admin" in the GUI somehow does not have any of those privs then they will not get the notices.
I would have thought there would be a lot of other stuff that "admin" cannot do if it is removed from the "admins" group.
Ditto, but that VM had apparently been broken in that way for some time and I never noticed until this morning when I expected to see a notice and it wasn't there. Definitely curious.
- Status changed from Assigned to Feedback
- Status changed from Feedback to Resolved
Works well now as far as I can see.
- Target version changed from 2.4.0 to 2.3.3
Also available in: Atom
PDF