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

Cannot remove missing permissions from post/page #1182

Closed
mikaellindemann opened this issue May 20, 2020 · 0 comments
Closed

Cannot remove missing permissions from post/page #1182

mikaellindemann opened this issue May 20, 2020 · 0 comments
Assignees
Milestone

Comments

@mikaellindemann
Copy link
Contributor

Ongoing development of a website can obsolete and remove permissions used to protect single pages or posts in Piranha.

Issue

If a permission is removed from the Piranha app, the permissions picker in the page/post settings is no longer available, even though the page/post is protected by said permission.

Therefore it is impossible to remove the required permission, while it still fails the user accessing the page with either Policy not found (in case the cleanup also removed the AspNetCore Authorization Policy, or just unauthorized if the policy is there, but the user does not have it.

Suggested solutions

  1. Automatically disregard unknown permissions (potentially on next publish of the page, if it makes it easier with caching and database updates).
  2. Show applied permissions in the permission picker, even though they no longer exist in the code. This allows the editor to remove the permission as a requirement for the page.
@tidyui tidyui added this to the Version 8.3 milestone May 21, 2020
@tidyui tidyui self-assigned this May 22, 2020
@tidyui tidyui closed this as completed in 771a62b May 22, 2020
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

2 participants