feat: improve code

apache · Mar 21, 2022 · 610d83a · 610d83a
1 parent c7c630b
commit 610d83a
Showing 1 changed file with 8 additions and 7 deletions.
diff --git a/hugegraph-core/src/main/java/com/baidu/hugegraph/security/HugeSecurityManager.java b/hugegraph-core/src/main/java/com/baidu/hugegraph/security/HugeSecurityManager.java
@@ -127,15 +127,15 @@ public class HugeSecurityManager extends SecurityManager {
             ImmutableSet.of("newSecurityException")
     );
 
-    private static final Set<String> ignoreCheck = new CopyOnWriteArraySet<>();
+    private static final Set<String> ignoreCheckedClasses = new CopyOnWriteArraySet<>();
 
-    public static void addIgnoreCheck(String clazz) {
+    public static void ignoreCheckedClass(String clazz) {
         if (callFromGremlin()) {
             throw newSecurityException(
                   "Not allowed to add ignore check via Gremlin");
         }
 
-        ignoreCheck.add(clazz);
+        ignoreCheckedClasses.add(clazz);
     }
 
     @Override
@@ -179,7 +179,7 @@ public void checkAccess(Thread thread) {
         if (callFromGremlin() && !callFromCaffeine() &&
             !callFromAsyncTasks() && !callFromEventHubNotify() &&
             !callFromBackendThread() && !callFromBackendHbase() &&
-            !callFromRaft() && !callFromSofaRpc() && !callFromIgnore()) {
+            !callFromRaft() && !callFromSofaRpc() && !callFromIgnoreCheckedClass()) {
             throw newSecurityException(
                   "Not allowed to access thread via Gremlin");
         }
@@ -191,7 +191,8 @@ public void checkAccess(ThreadGroup threadGroup) {
         if (callFromGremlin() && !callFromCaffeine() &&
             !callFromAsyncTasks() && !callFromEventHubNotify() &&
             !callFromBackendThread() && !callFromBackendHbase() &&
-            !callFromRaft() && !callFromSofaRpc() && !callFromIgnore()) {
+            !callFromRaft() && !callFromSofaRpc() &&
+            !callFromIgnoreCheckedClass()) {
             throw newSecurityException(
                   "Not allowed to access thread group via Gremlin");
         }
@@ -487,8 +488,8 @@ private static boolean callFromNewSecurityException() {
         return callFromMethods(NEW_SECURITY_EXCEPTION);
     }
 
-    private static boolean callFromIgnore() {
-        return callFromWorkerWithClass(ignoreCheck);
+    private static boolean callFromIgnoreCheckedClass() {
+        return callFromWorkerWithClass(ignoreCheckedClasses);
     }
 
     private static boolean callFromWorkerWithClass(Set<String> classes) {