?
avatar dautrich
dautrich
29 Jan 2018

I added a new user to standard user group Administrator. When I log in with this user, menu item System Information in panel System is shown (see 1.jpg). Clicking this item, an error 403 is thrown (see 2.jpg).

Steps to reproduce the issue

Add user to standard user group "Administrator".
Log in with this user.
Click on "System information" under "System".

Expected result

System information is displayed.

Actual result

Error 403 (Not authorized).

System information (as much as possible)

Joomla 3.8.3 with only a small number of extensions (Akeeba Backup, JCE, OSMap, Phoca Maps, SIGPlus; all updated)

Additional comments

In my understanding, an Administrator should be able to see the System Information. A Manager shouldn't.

screen shot 2018-01-29 at 15 03 50
screen shot 2018-01-29 at 15 03 50

avatar dautrich dautrich - open - 29 Jan 2018
avatar joomla-cms-bot joomla-cms-bot - labeled - 29 Jan 2018
avatar mbabker
mbabker - comment - 29 Jan 2018

This is by design - https://github.com/joomla/joomla-cms/blob/3.8.3/administrator/components/com_admin/views/sysinfo/view.html.php#L71

The view checks for global super user permissions to see this screen as it exposes potentially sensitive information about the server setup.

avatar dautrich
dautrich - comment - 29 Jan 2018

I got your point. But if it's designed like this, the menu item "System Information" shouldn't be displayed for an Administrator, but only for a Super User.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19478.

avatar mbabker
mbabker - comment - 29 Jan 2018

This sounds vaguely like another of the issues we have where admin menu items need to be shown/hidden based on ACL (or in this case an ACL level other than the core.manage usually used) and not viewing levels, for which there seem to already be a bunch of workarounds in https://github.com/joomla/joomla-cms/blob/3.8.3/administrator/modules/mod_menu/menu.php#L234 for.

avatar franz-wohlkoenig franz-wohlkoenig - change - 29 Jan 2018
Status New Discussion
avatar Quy Quy - change - 16 Feb 2018
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2018-02-16 00:05:57
Closed_By Quy
avatar joomla-cms-bot joomla-cms-bot - change - 16 Feb 2018
Closed_By Quy joomla-cms-bot
avatar joomla-cms-bot joomla-cms-bot - close - 16 Feb 2018
avatar Quy
Quy - comment - 16 Feb 2018

See PR #19697


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19478.

avatar joomla-cms-bot
joomla-cms-bot - comment - 16 Feb 2018

Set to "closed" on behalf of @Quy by The JTracker Application at issues.joomla.org/joomla-cms/19478

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 16 Feb 2018

@dautrich please test #19697

avatar dautrich
dautrich - comment - 16 Feb 2018

@franz-wohlkoenig I have successfully tested #19697 (s. screenshot)
result_19697

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 16 Feb 2018

@dautrich please mark your Test as successfully:

  • open Issue Tracker
  • Login with your github-Account
  • Click on blue "Test this"-Button above Authors-Picture
  • mark your Test as successfully
  • hit "submit test result"
avatar dautrich
dautrich - comment - 16 Feb 2018
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 16 Feb 2018

Thanks @dautrich for Test. Now the Pull Request need a second successfully Test before Release Lead decide if it gets merged.

Add a Comment

Login with GitHub to post a comment