This is an automated email from the ASF dual-hosted git repository.

yuqi1129 pushed a commit to branch branch-1.3
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to refs/heads/branch-1.3 by this push:
     new 0fb858a117 [Cherry-pick to branch-1.3] [MINOR] fix(server): Suppress 
verbose JCasbin authorization logs (#13270) (#13280)
0fb858a117 is described below

commit 0fb858a117e512e264da438acfb279ad568d40cd
Author: github-actions[bot] 
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Thu Sep 17 23:11:25 2026 +0800

    [Cherry-pick to branch-1.3] [MINOR] fix(server): Suppress verbose JCasbin 
authorization logs (#13270) (#13280)
    
    **Cherry-pick Information:**
    - Original commit: 11bd60d7e3e5a862a4b58fe6e860edc34e642679
    - Target branch: `branch-1.3`
    - Status: ✅ Clean cherry-pick (no conflicts)
    
    Co-authored-by: roryqi <[email protected]>
---
 conf/log4j2.properties.template                         |  4 ++++
 dev/charts/gravitino/resources/config/log4j2.properties |  4 ++++
 dev/charts/gravitino/tests/configmap_test.yaml          | 10 ++++++++++
 dev/charts/gravitino/values.yaml                        |  3 +++
 4 files changed, 21 insertions(+)

diff --git a/conf/log4j2.properties.template b/conf/log4j2.properties.template
index ff27eb3e0b..1952d6ae06 100644
--- a/conf/log4j2.properties.template
+++ b/conf/log4j2.properties.template
@@ -106,6 +106,10 @@ logger.rest.level = warn
 logger.rest.appenderRef.rolling.ref = fileLogger
 logger.rest.additivity = false
 
+# Suppress verbose JCasbin authorization decision logs by default
+logger.jcasbin.name = org.casbin.jcasbin
+logger.jcasbin.level = warn
+
 # Configure root logger
 rootLogger.level = info
 rootLogger.appenderRef.rolling.ref = fileLogger
diff --git a/dev/charts/gravitino/resources/config/log4j2.properties 
b/dev/charts/gravitino/resources/config/log4j2.properties
index 20575d9333..e6101c9517 100644
--- a/dev/charts/gravitino/resources/config/log4j2.properties
+++ b/dev/charts/gravitino/resources/config/log4j2.properties
@@ -47,6 +47,10 @@ appender.rolling.strategy.delete.ifLastModified.type = {{ 
.Values.log4j2Properti
 # Delete all files older than 30 days
 appender.rolling.strategy.delete.ifLastModified.age = {{ 
.Values.log4j2Properties.rollingAppenderStrategyDeleteIfLastModifiedAge | 
default "30d" }}
 
+# Suppress verbose JCasbin authorization decision logs by default
+logger.jcasbin.name = org.casbin.jcasbin
+logger.jcasbin.level = {{ .Values.log4j2Properties.jcasbinLoggerLevel | 
default "warn" }}
+
 # Configure root logger
 rootLogger.level = {{ .Values.log4j2Properties.rootLoggerLevel | default 
"info" }}
 rootLogger.appenderRef.rolling.ref = {{ 
.Values.log4j2Properties.rootLoggerAppenderRefRollingRef | default "fileLogger" 
}}
diff --git a/dev/charts/gravitino/tests/configmap_test.yaml 
b/dev/charts/gravitino/tests/configmap_test.yaml
index d34ac33e27..d0d9964d86 100644
--- a/dev/charts/gravitino/tests/configmap_test.yaml
+++ b/dev/charts/gravitino/tests/configmap_test.yaml
@@ -49,6 +49,12 @@ tests:
       - matchRegex:
           path: data["log4j2.properties"]
           pattern: "rootLogger\\.level = info"
+      - matchRegex:
+          path: data["log4j2.properties"]
+          pattern: "logger\\.jcasbin\\.name = org\\.casbin\\.jcasbin"
+      - matchRegex:
+          path: data["log4j2.properties"]
+          pattern: "logger\\.jcasbin\\.level = warn"
 
   - it: renders customized Gravitino and log4j configuration
     release:
@@ -63,6 +69,7 @@ tests:
         gravitino.custom.setting: "{{ .Values.entity.store }}"
       log4j2Properties:
         rootLoggerLevel: debug
+        jcasbinLoggerLevel: debug
       additionalLog4j2Properties:
         logger.custom.name: org.apache.gravitino.Custom
     asserts:
@@ -81,6 +88,9 @@ tests:
       - matchRegex:
           path: data["log4j2.properties"]
           pattern: "rootLogger\\.level = debug"
+      - matchRegex:
+          path: data["log4j2.properties"]
+          pattern: "logger\\.jcasbin\\.level = debug"
       - matchRegex:
           path: data["log4j2.properties"]
           pattern: "logger\\.custom\\.name = org\\.apache\\.gravitino\\.Custom"
diff --git a/dev/charts/gravitino/values.yaml b/dev/charts/gravitino/values.yaml
index 157bc27cf8..a08a33d6da 100644
--- a/dev/charts/gravitino/values.yaml
+++ b/dev/charts/gravitino/values.yaml
@@ -472,6 +472,9 @@ log4j2Properties: {}
   ## Delete all files older than 30 days
   # rollingAppenderStrategyDeleteIfLastModifiedAge: 30d
 
+  ## JCasbin logger
+  # jcasbinLoggerLevel: warn
+
   ## Configure root logger
   # rootLoggerLevel: info
   # rootLoggerAppenderRefRollingRef: fileLogger

Reply via email to