Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[4.1] Split up com_users permissions into sections #24299

Open
4 tasks
SniperSister opened this issue Mar 22, 2019 · 8 comments
Open
4 tasks

[4.1] Split up com_users permissions into sections #24299

SniperSister opened this issue Mar 22, 2019 · 8 comments
Assignees

Comments

@SniperSister
Copy link
Contributor

Is your feature request related to a problem? Please describe.

Right now, there's just a global "edit" permission for com_users, meaning that if you want to allow someone to edit users, you also have to allow them to edit groups and view levels, which obviously isn't always wanted as a security implication in terms of ACL is associated to that.

Describe the solution you'd like

We should split up the "com_users" ACL right into multiple rights, just like we do in the rest of the CMS:

Tasks:

  • split up com_users into com_users.users, com_users.groups, com_users.viewlevels
  • adjust permission level checks (so far checking for com_users) throughout the system
  • update default permissions to make sure that groups and viewlevels is allowed for Super Admins only
  • figure out a 3.x to 4.x migration path

Additional context

JSST had an internal discussion about this topic, but as the current behavior is "by design" and any change would be a b/c break, we decided to open a ticket in the public tracker instead of handling it in a security release.

@brianteeman
Copy link
Contributor

Is this something you will be contributing?

@brianteeman
Copy link
Contributor

or should this be closed?

@brianteeman
Copy link
Contributor

@SniperSister any update?

@SniperSister
Copy link
Contributor Author

Drowning in work so didn't had time to tackle that issue, but it's definitely on my todo list

@alexandreelise
Copy link
Contributor

Nice feature to add. @SniperSister


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

@brianteeman
Copy link
Contributor

Thsi should probably be retagged for 4.1

@Quy Quy added PR-4.1-dev and removed J4 Issue labels Dec 23, 2020
@joomla-cms-bot joomla-cms-bot changed the title [4.0] Split up com_users permissions into sections [4.1] Split up com_users permissions into sections Aug 24, 2021
@jwaisner
Copy link
Member

Updated to retag (without a label at the moment) to revisit for 4.1


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

@brianteeman
Copy link
Contributor

please add the new feature and j4 issue label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants