You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by zh...@apache.org on 2020/05/08 12:34:17 UTC

[pulsar] 31/38: change log level to debug for bouncy castle load (#6860)

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

zhaijia pushed a commit to branch branch-2.5
in repository https://gitbox.apache.org/repos/asf/pulsar.git

commit 4063f79819fcc0f44d9979b84208d290560d4e9f
Author: Jia Zhai <zh...@apache.org>
AuthorDate: Wed May 6 20:31:31 2020 +0800

    change log level to debug for bouncy castle load (#6860)
    
    change log level to debug to avoid annoying.
    
    (cherry picked from commit 4e6b2f09a08880faf8e8eb5753285ced0b3bf519)
---
 .../main/java/org/apache/pulsar/common/util/SearchBcNarUtils.java | 6 ++++--
 .../main/java/org/apache/pulsar/common/util/SecurityUtility.java  | 8 ++++++--
 2 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/pulsar-common/src/main/java/org/apache/pulsar/common/util/SearchBcNarUtils.java b/pulsar-common/src/main/java/org/apache/pulsar/common/util/SearchBcNarUtils.java
index e3d4f88..0dfa469 100644
--- a/pulsar-common/src/main/java/org/apache/pulsar/common/util/SearchBcNarUtils.java
+++ b/pulsar-common/src/main/java/org/apache/pulsar/common/util/SearchBcNarUtils.java
@@ -89,8 +89,10 @@ public class SearchBcNarUtils {
                         }
 
                         Provider provider = ((BCLoader) loader).getProvider();
-                        log.info("Found Bouncy Castle loader {} from {}, provider: {}",
-                                loader.getClass().getCanonicalName(), path, provider.getName());
+                        if (log.isDebugEnabled()) {
+                            log.debug("Found Bouncy Castle loader {} from {}, provider: {}",
+                                    loader.getClass().getCanonicalName(), path, provider.getName());
+                        }
                         loadFuture.complete(provider);
                     } catch (Throwable t) {
                         log.error("Failed to load Bouncy Castle Provider ", t);
diff --git a/pulsar-common/src/main/java/org/apache/pulsar/common/util/SecurityUtility.java b/pulsar-common/src/main/java/org/apache/pulsar/common/util/SecurityUtility.java
index d3a2615..648e9f2 100644
--- a/pulsar-common/src/main/java/org/apache/pulsar/common/util/SecurityUtility.java
+++ b/pulsar-common/src/main/java/org/apache/pulsar/common/util/SecurityUtility.java
@@ -87,7 +87,9 @@ public class SecurityUtility {
             Provider provider = Security.getProvider(BC) != null
                     ? Security.getProvider(BC)
                     : Security.getProvider(BC_FIPS);
-            log.info("Already instantiated Bouncy Castle provider {}", provider.getName());
+            if (log.isDebugEnabled()) {
+                log.debug("Already instantiated Bouncy Castle provider {}", provider.getName());
+            }
             return provider;
         }
 
@@ -125,7 +127,9 @@ public class SecurityUtility {
 
         Provider provider = (Provider) clazz.newInstance();
         Security.addProvider(provider);
-        log.info("Found and Instantiated Bouncy Castle provider in classpath {}", provider.getName());
+        if (log.isDebugEnabled()) {
+            log.debug("Found and Instantiated Bouncy Castle provider in classpath {}", provider.getName());
+        }
         return provider;
     }