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.

Sign up for GitHub

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

Jump to bottom

Editing current user should prevent changes of account status #4567

Closed
TheWitness opened this issue Feb 23, 2022 · 0 comments
Closed

Editing current user should prevent changes of account status #4567

TheWitness opened this issue Feb 23, 2022 · 0 comments
Labels
bug Undesired behaviour resolved A fixed issue
Milestone
v1.2.20

Comments

@TheWitness
Copy link
Member

Describe the bug

From the GUI perspective, Cacti appears to allow the removal of the currently logged in user and it should not.

Expected behavior

Cacti should block harmful actions from the GUI or the backend business rules.
@TheWitness TheWitness added the bug Undesired behaviour label Feb 23, 2022
@TheWitness TheWitness added this to the v1.2.20 milestone Feb 23, 2022
TheWitness added a commit that referenced this issue Feb 24, 2022
@TheWitness
Fixing Issues #4568 and #4567 - User Management and Auth Issues
d3294bb 
- Cacti should disallow deleting the currently logged in user
- Cacti incorrectly uses the user_auth_cache from identical username from the wrong realm
- This second issue may require further work, but no in the 1.2.20 release
@TheWitness TheWitness added the resolved A fixed issue label Feb 24, 2022
@netniV netniV changed the title Cacti should disallow deleting of the currently logged in user Editing current user should prevent changes of account status Apr 3, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Dec 2, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug Undesired behaviour resolved A fixed issue
Projects
None yet
Milestone
v1.2.20
Development

No branches or pull requests
1 participant
@TheWitness