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

Logged in admins do not lose admin status when it is changed + deleted users aren't kicked out #186

Open
qy21gafy opened this issue Jan 24, 2023 · 2 comments
Assignees
Labels
bug Something isn't working Est. Size = 3 Medium Size

Comments

@qy21gafy
Copy link
Collaborator

qy21gafy commented Jan 24, 2023

Describe the bug
When an admin removes the admin status of itself, the status does not update until he logs out and logs in again. This also happens when an admin updates the status of another admin that is currently still logged in -> could pose large security problem. Deleted user's also aren't kicked out and can still book desks.

To Reproduce
Steps to reproduce the behavior:

  1. Go to user management overview as an admin using the production server
  2. Click on the admin checkbox of the logged in admin
  3. The admin should now no longer be an admin and should not be able to access the user management page
  4. However the logged in user can now just reclaim his admin priviledges

Expected behavior
The change in admin status should immediately change so that logged in previous admins can not just reclaim their admin status immediately. Deleted user's should also be kicked out when their account is deleted.

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

  • OS: Windows 10
  • Browser: Chrome
  • Version [e.g. 22]

Smartphone (please complete the following information):

  • Device: -
  • OS: -
  • Browser: -
  • Version: -

Additional context
Add any other context about the problem here.

@qy21gafy qy21gafy added the bug Something isn't working label Jan 24, 2023
@qy21gafy qy21gafy moved this to Product Backlog in amos2022ws05-feature-backlog Jan 24, 2023
@xilef45
Copy link
Collaborator

xilef45 commented Jan 25, 2023

That actually works. He only has to login again. So he can undo this action.

@xilef45 xilef45 linked a pull request Jan 31, 2023 that will close this issue
@xilef45 xilef45 removed a link to a pull request Jan 31, 2023
@xilef45 xilef45 reopened this Jan 31, 2023
@qy21gafy qy21gafy changed the title Logged in admins do not lose admin status when it is changed Logged in admins do not lose admin status when it is changed + deleted users aren't kicked out Jan 31, 2023
@qy21gafy qy21gafy added the Est. Size = 3 Medium Size label Feb 1, 2023
@qy21gafy qy21gafy moved this from Product Backlog to Sprint backlog in amos2022ws05-feature-backlog Feb 1, 2023
@jantiegges jantiegges self-assigned this Feb 2, 2023
@jantiegges jantiegges moved this from Sprint backlog to In progress in amos2022ws05-feature-backlog Feb 6, 2023
@jantiegges
Copy link
Collaborator

One can not change it's own access rights

@jantiegges jantiegges moved this from In progress to Sprint backlog in amos2022ws05-feature-backlog Feb 7, 2023
@jantiegges jantiegges moved this from Sprint backlog to In progress in amos2022ws05-feature-backlog Feb 7, 2023
@qy21gafy qy21gafy moved this from In progress to Product Backlog in amos2022ws05-feature-backlog Feb 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Est. Size = 3 Medium Size
Projects
Status: Product Backlog
Development

No branches or pull requests

3 participants