-
Notifications
You must be signed in to change notification settings - Fork 120
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
CODENVY-1711: Grant permissions to admins for predefined stacks #1768
Conversation
@@ -86,8 +103,66 @@ public void filter(GenericResourceMethod genericResourceMethod, Object[] argumen | |||
throw new ForbiddenException("The user does not have permission to perform this operation"); | |||
} | |||
|
|||
if (currentSubject.hasPermission(SystemDomain.DOMAIN_ID, stackId, SystemDomain.MANAGE_SYSTEM_ACTION) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
currentSubject.hasPermission(SystemDomain.DOMAIN_ID, null, SystemDomain.MANAGE_SYSTEM_ACTION)
return permissionsManager.getByInstance(DOMAIN_ID, stackId, 25, 0); | ||
} | ||
if (!permissionsPage.hasNextPage()) { | ||
return null; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think that it is good idea to return null here
…te and delete to predefined stacks Signed-off-by: Mykola Morhun <mmorhun@codenvy.com>
3eaf098
to
22664ca
Compare
ci-build |
Add yourself as author to StackPermissionsFilter please |
Signed-off-by: Mykola Morhun <mmorhun@codenvy.com>
These will appear in Swagger docs, right? |
@bmicklea no, it will just perform action instead of error with permission denied message in case of predefined stack. |
What does this PR do?
Allows to users with
manageSystem
permission perform edit and delete operation with predefined (i.e. public for all users) stack.What issues does this PR fix or reference?
#1711
Changelog
Permissions granted to admins to perform
update
anddelete
action with predefined stacks.Release Notes
Added ability to admins with
manageSystem
permission performupdate
anddelete
action with predefined stacks.Docs PR
N/A