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

Passed user objects are not reloaded form database during CRUD operations #1155

Open
helkv opened this issue Sep 6, 2019 · 0 comments
Open
Labels
bug Issue found during QA testphase (qa-imeji)

Comments

@helkv
Copy link
Contributor

helkv commented Sep 6, 2019

Testserver: qa imeji
Browser: ff
Version: 4.6 - build date 2019-08-30 12:57:36
User: Admin & User

Passed user objects are not reloaded form database when CRUD operations are executed.

When authorization is checked during a CRUD operation the grants of the passed session user object are used.
Outdated grants are used for the authorization check, if the user grants have changed but the session user was not updated.

Actions:

  1. Login as user
  2. Login as admin [parallel in another browser/session]
  3. Open edit item view of an item (as user)
  4. Remove the edit grant from the user for this item/collection (as admin)
  5. Edit and save the item (as user)
  6. The user can still save the changes and the item gets updated, although the user has no more rights to edit the item

Expected (result): User grants should be reloaded from DB before/in the authorization-check => user can make changes only on the basis of his current DB grants, even if his session grants are outdated.

@helkv helkv added the bug Issue found during QA testphase (qa-imeji) label Sep 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue found during QA testphase (qa-imeji)
Projects
None yet
Development

No branches or pull requests

1 participant