kafka git commit: KAFKA-5294: PlainSaslServerFactory should allow a null Map

2017-05-22 Thread rsivaram
Repository: kafka
Updated Branches:
  refs/heads/0.11.0 6d04e412c -> ef88ee504


KAFKA-5294: PlainSaslServerFactory should allow a null Map

If props is null, use POLICY_NOPLAINTEXT default value: false

Author: Mickael Maison 

Reviewers: Ismael Juma , Rajini Sivaram 


Closes #3105 from mimaison/KAFKA-5294

(cherry picked from commit d3f1407f7bc95b823516e5a1bdaa23c717b6354e)
Signed-off-by: Rajini Sivaram 


Project: http://git-wip-us.apache.org/repos/asf/kafka/repo
Commit: http://git-wip-us.apache.org/repos/asf/kafka/commit/ef88ee50
Tree: http://git-wip-us.apache.org/repos/asf/kafka/tree/ef88ee50
Diff: http://git-wip-us.apache.org/repos/asf/kafka/diff/ef88ee50

Branch: refs/heads/0.11.0
Commit: ef88ee5045ea791d3208c3dcbc8723c4b2092262
Parents: 6d04e41
Author: Mickael Maison 
Authored: Mon May 22 11:00:30 2017 +0100
Committer: Rajini Sivaram 
Committed: Mon May 22 11:02:14 2017 +0100

--
 .../org/apache/kafka/common/security/plain/PlainSaslServer.java | 1 +
 1 file changed, 1 insertion(+)
--


http://git-wip-us.apache.org/repos/asf/kafka/blob/ef88ee50/clients/src/main/java/org/apache/kafka/common/security/plain/PlainSaslServer.java
--
diff --git 
a/clients/src/main/java/org/apache/kafka/common/security/plain/PlainSaslServer.java
 
b/clients/src/main/java/org/apache/kafka/common/security/plain/PlainSaslServer.java
index df59820..6fe3b25 100644
--- 
a/clients/src/main/java/org/apache/kafka/common/security/plain/PlainSaslServer.java
+++ 
b/clients/src/main/java/org/apache/kafka/common/security/plain/PlainSaslServer.java
@@ -160,6 +160,7 @@ public class PlainSaslServer implements SaslServer {
 
 @Override
 public String[] getMechanismNames(Map props) {
+if (props == null) return new String[]{PLAIN_MECHANISM};
 String noPlainText = (String) props.get(Sasl.POLICY_NOPLAINTEXT);
 if ("true".equals(noPlainText))
 return new String[]{};



kafka git commit: KAFKA-5294: PlainSaslServerFactory should allow a null Map

2017-05-22 Thread rsivaram
Repository: kafka
Updated Branches:
  refs/heads/trunk 495836a4a -> d3f1407f7


KAFKA-5294: PlainSaslServerFactory should allow a null Map

If props is null, use POLICY_NOPLAINTEXT default value: false

Author: Mickael Maison 

Reviewers: Ismael Juma , Rajini Sivaram 


Closes #3105 from mimaison/KAFKA-5294


Project: http://git-wip-us.apache.org/repos/asf/kafka/repo
Commit: http://git-wip-us.apache.org/repos/asf/kafka/commit/d3f1407f
Tree: http://git-wip-us.apache.org/repos/asf/kafka/tree/d3f1407f
Diff: http://git-wip-us.apache.org/repos/asf/kafka/diff/d3f1407f

Branch: refs/heads/trunk
Commit: d3f1407f7bc95b823516e5a1bdaa23c717b6354e
Parents: 495836a
Author: Mickael Maison 
Authored: Mon May 22 11:00:30 2017 +0100
Committer: Rajini Sivaram 
Committed: Mon May 22 11:00:30 2017 +0100

--
 .../org/apache/kafka/common/security/plain/PlainSaslServer.java | 1 +
 1 file changed, 1 insertion(+)
--


http://git-wip-us.apache.org/repos/asf/kafka/blob/d3f1407f/clients/src/main/java/org/apache/kafka/common/security/plain/PlainSaslServer.java
--
diff --git 
a/clients/src/main/java/org/apache/kafka/common/security/plain/PlainSaslServer.java
 
b/clients/src/main/java/org/apache/kafka/common/security/plain/PlainSaslServer.java
index df59820..6fe3b25 100644
--- 
a/clients/src/main/java/org/apache/kafka/common/security/plain/PlainSaslServer.java
+++ 
b/clients/src/main/java/org/apache/kafka/common/security/plain/PlainSaslServer.java
@@ -160,6 +160,7 @@ public class PlainSaslServer implements SaslServer {
 
 @Override
 public String[] getMechanismNames(Map props) {
+if (props == null) return new String[]{PLAIN_MECHANISM};
 String noPlainText = (String) props.get(Sasl.POLICY_NOPLAINTEXT);
 if ("true".equals(noPlainText))
 return new String[]{};