You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kafka.apache.org by rs...@apache.org on 2018/02/14 17:53:53 UTC

[kafka] branch trunk updated: MINOR: Update log condition in JassContext.loadServerContext (#4566)

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

rsivaram pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/kafka.git


The following commit(s) were added to refs/heads/trunk by this push:
     new 36b0030  MINOR: Update log condition in JassContext.loadServerContext (#4566)
36b0030 is described below

commit 36b00300f5a9117abb4e8570c1714b3023a0ab01
Author: Manikumar Reddy O <ma...@gmail.com>
AuthorDate: Wed Feb 14 23:23:49 2018 +0530

    MINOR: Update log condition in JassContext.loadServerContext (#4566)
---
 clients/src/main/java/org/apache/kafka/common/security/JaasContext.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/clients/src/main/java/org/apache/kafka/common/security/JaasContext.java b/clients/src/main/java/org/apache/kafka/common/security/JaasContext.java
index 46db345..6afed55 100644
--- a/clients/src/main/java/org/apache/kafka/common/security/JaasContext.java
+++ b/clients/src/main/java/org/apache/kafka/common/security/JaasContext.java
@@ -65,7 +65,7 @@ public class JaasContext {
         String globalContextName = GLOBAL_CONTEXT_NAME_SERVER;
         String listenerContextName = listenerName.value().toLowerCase(Locale.ROOT) + "." + GLOBAL_CONTEXT_NAME_SERVER;
         Password jaasConfigArgs = (Password) configs.get(mechanism.toLowerCase(Locale.ROOT) + "." + SaslConfigs.SASL_JAAS_CONFIG);
-        if (jaasConfigArgs == null && configs.containsKey(SaslConfigs.SASL_JAAS_CONFIG))
+        if (jaasConfigArgs == null && configs.get(SaslConfigs.SASL_JAAS_CONFIG) != null)
             LOG.warn("Server config {} should be prefixed with SASL mechanism name, ignoring config", SaslConfigs.SASL_JAAS_CONFIG);
         return load(Type.SERVER, listenerContextName, globalContextName, jaasConfigArgs);
     }

-- 
To stop receiving notification emails like this one, please contact
rsivaram@apache.org.