[HOTFIX] Fix commented

jongyoul · Feb 7, 2024 · 982e64b · 982e64b
1 parent 7cdef8a
commit 982e64b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/zeppelin-interpreter/src/main/java/org/apache/zeppelin/conf/ZeppelinConfiguration.java b/zeppelin-interpreter/src/main/java/org/apache/zeppelin/conf/ZeppelinConfiguration.java
@@ -597,7 +597,7 @@ public String getShiroPath() {
   }
 
   public boolean isAuthenticationEnabled() {
-    return StringUtils.EMPTY != getShiroPath();
+    return !StringUtils.isBlank(getShiroPath());
   }
 
   public String getInterpreterRemoteRunnerPath() {

diff --git a/zeppelin-server/src/main/java/org/apache/zeppelin/service/NotebookService.java b/zeppelin-server/src/main/java/org/apache/zeppelin/service/NotebookService.java
@@ -913,7 +913,7 @@ public void updateNote(String noteId,
               return null;
             }
 
-            if (context.getUserAndRoles().containsAll(requestCronRoles)) {
+            if (!context.getUserAndRoles().containsAll(requestCronRoles)) {
               LOGGER.error("Wrong cronExecutingRoles: {}", requestCronRoles);
               callback.onFailure(new IllegalArgumentException(requestCronRoles.toString()), context);
               return null;