You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by ro...@apache.org on 2020/05/05 11:25:47 UTC

[qpid-jms] branch master updated: QPIDJMS-501: lower levels for SASL mech selection logging

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

robbie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/qpid-jms.git


The following commit(s) were added to refs/heads/master by this push:
     new 6769f5e  QPIDJMS-501: lower levels for SASL mech selection logging
6769f5e is described below

commit 6769f5e51694271ec8c019a02599b66d99a5575e
Author: Robbie Gemmell <ro...@apache.org>
AuthorDate: Tue May 5 12:25:14 2020 +0100

    QPIDJMS-501: lower levels for SASL mech selection logging
---
 .../java/org/apache/qpid/jms/sasl/SaslMechanismFinder.java     | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/qpid-jms-client/src/main/java/org/apache/qpid/jms/sasl/SaslMechanismFinder.java b/qpid-jms-client/src/main/java/org/apache/qpid/jms/sasl/SaslMechanismFinder.java
index 10b2cdc..afc5889 100644
--- a/qpid-jms-client/src/main/java/org/apache/qpid/jms/sasl/SaslMechanismFinder.java
+++ b/qpid-jms-client/src/main/java/org/apache/qpid/jms/sasl/SaslMechanismFinder.java
@@ -80,15 +80,15 @@ public class SaslMechanismFinder {
 
                 boolean mechConfigured = mechRestrictions != null && mechRestrictions.contains(remoteMechanism);
                 if (mechRestrictions != null && !mechConfigured) {
-                    LOG.debug("Skipping {} mechanism because it is not in the configured mechanisms restriction set", remoteMechanism);
+                    LOG.trace("Skipping {} mechanism because it is not in the configured mechanisms restriction set", remoteMechanism);
                 } else if (mech.isApplicable(username, password, localPrincipal)) {
                     if (mech.isEnabledByDefault() || mechConfigured) {
                         found.add(mech);
                     } else {
-                        LOG.debug("Skipping {} mechanism as it must be explicitly enabled in the configured sasl mechanisms", mech);
+                        LOG.trace("Skipping {} mechanism as it must be explicitly enabled in the configured sasl mechanisms", mech);
                     }
                 } else {
-                    LOG.debug("Skipping {} mechanism because the available credentials are not sufficient", mech);
+                    LOG.trace("Skipping {} mechanism because the available credentials are not sufficient", mech);
                 }
             }
         }
@@ -103,7 +103,7 @@ public class SaslMechanismFinder {
                 "No supported mechanism, or none usable with the available credentials. Server offered: " + remoteMechanismNames);
         }
 
-        LOG.info("Best match for SASL auth was: {}", match);
+        LOG.debug("Best match for SASL auth was: {}", match);
 
         return match;
     }
@@ -129,7 +129,7 @@ public class SaslMechanismFinder {
         try {
             factory = MECHANISM_FACTORY_FINDER.newInstance(name);
         } catch (ResourceNotFoundException rnfe) {
-            LOG.debug("Unknown SASL mechanism: [" + name + "]");
+            LOG.trace("Unknown SASL mechanism: [" + name + "]");
         } catch (Exception e) {
             LOG.warn("Caught exception while finding factory for SASL mechanism {}: {}", name, e.getMessage());
         }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org