Project

General

Profile

Actions
Copy link

Feature #7051

closed
PD PD

Allow control of what users can view and/or clear notices

Feature #7051: Allow control of what users can view and/or clear notices

Added by Phillip Davis over 9 years ago. Updated over 9 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Phillip Davis
Category:
Dashboard
Target version:
2.3.3
Start date:
12/29/2016
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
Release Notes:

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

RB Updated by Renato Botelho over 9 years ago Actions
Copy link
 #2

  • Status changed from New to Feedback
  • % Done changed from 0 to 100

PR has been merged, thanks!

JP Updated by Jim Pingle over 9 years ago Actions
Copy link
 #3

  • 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.

JP Updated by Jim Pingle over 9 years ago Actions
Copy link
 #4

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.

PD Updated by Phillip Davis over 9 years ago Actions
Copy link
 #5

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.

JP Updated by Jim Pingle over 9 years ago Actions
Copy link
 #6

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.

PD Updated by Phillip Davis over 9 years ago Actions
Copy link
 #7

This should fix it:
https://github.com/pfsense/pfsense/pull/3359
assuming it should be "fixed"

JP Updated by Jim Pingle over 9 years ago Actions
Copy link
 #8

  • Status changed from Assigned to Feedback

PR merged

JP Updated by Jim Pingle over 9 years ago Actions
Copy link
 #9

  • Status changed from Feedback to Resolved

Works well now as far as I can see.

JP Updated by Jim Pingle over 9 years ago Actions
Copy link
 #10

  • Target version changed from 2.4.0 to 2.3.3
Actions
Copy link

Also available in: Atom