You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by li...@apache.org on 2017/04/21 04:06:03 UTC

kylin git commit: minor, mark KYLIN-1664 in code

Repository: kylin
Updated Branches:
  refs/heads/2.0.x 3fb74fe49 -> 79aa5456e


minor, mark KYLIN-1664 in code


Project: http://git-wip-us.apache.org/repos/asf/kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/79aa5456
Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/79aa5456
Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/79aa5456

Branch: refs/heads/2.0.x
Commit: 79aa5456eab48b036e33c483272aac8783ae5169
Parents: 3fb74fe
Author: Li Yang <li...@apache.org>
Authored: Fri Apr 21 12:05:57 2017 +0800
Committer: Li Yang <li...@apache.org>
Committed: Fri Apr 21 12:05:57 2017 +0800

----------------------------------------------------------------------
 .../src/main/java/org/apache/kylin/rest/service/AdminService.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kylin/blob/79aa5456/server-base/src/main/java/org/apache/kylin/rest/service/AdminService.java
----------------------------------------------------------------------
diff --git a/server-base/src/main/java/org/apache/kylin/rest/service/AdminService.java b/server-base/src/main/java/org/apache/kylin/rest/service/AdminService.java
index 66725dc..5c890f7 100644
--- a/server-base/src/main/java/org/apache/kylin/rest/service/AdminService.java
+++ b/server-base/src/main/java/org/apache/kylin/rest/service/AdminService.java
@@ -89,7 +89,7 @@ public class AdminService extends BasicService {
      *
      * @return
      */
-    // @PreAuthorize(Constant.ACCESS_HAS_ROLE_ADMIN)
+    // @PreAuthorize(Constant.ACCESS_HAS_ROLE_ADMIN)  // this is a critical security issue, see KYLIN-1664
     public String getConfigAsString() {
         logger.debug("Get Kylin Runtime Config");