[
https://issues.apache.org/jira/browse/KYLIN-3239?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16353215#comment-16353215
]
Guangyao Li commented on KYLIN-3239:
------------------------------------
There are two permissions patterns "checkPermission" and "hasPermission" in the
file "AclEvaluate.Java".
The "checkPermission" method should check the permission of the current user
and doesn't need to return a value. This is a void method.
The "hasPermission" method is a boolean method, and should return a value about
"ture" or "false".
The refactoring covers the root file "AclEvaluate.Java" and some else files
using "checkPermission" or "hasPermission" method.
Now, the files using "checkPermission" or "hasPermission" method have
"ProjectController.Java, CubeService.Java, ModelService.Java,
ProjectService.Java".
> Refactor the ACL code about "checkPermission" and "hasPermission"
> -----------------------------------------------------------------
>
> Key: KYLIN-3239
> URL: https://issues.apache.org/jira/browse/KYLIN-3239
> Project: Kylin
> Issue Type: Improvement
> Components: Security
> Affects Versions: v2.2.0
> Reporter: Guangyao Li
> Priority: Minor
> Fix For: v2.3.0
>
>
> According to the design of "checkPermission" and "HasPermission",
> Refactoring the ACL code
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
