kafka-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ij...@apache.org
Subject kafka git commit: MINOR: Remove unnecessary options in SCRAM test jaas config
Date Fri, 13 Jan 2017 02:21:04 GMT
Repository: kafka
Updated Branches:
  refs/heads/trunk ad8ddf737 -> 2b6ba55c7


MINOR: Remove unnecessary options in SCRAM test jaas config

Author: Rajini Sivaram <rajinisivaram@googlemail.com>

Reviewers: Ismael Juma <ismael@juma.me.uk>

Closes #2353 from rajinisivaram/minor-scram


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

Branch: refs/heads/trunk
Commit: 2b6ba55c73b72fb3bfb6ebb2e2c8507cbacd5690
Parents: ad8ddf7
Author: Rajini Sivaram <rajinisivaram@googlemail.com>
Authored: Fri Jan 13 02:20:58 2017 +0000
Committer: Ismael Juma <ismael@juma.me.uk>
Committed: Fri Jan 13 02:20:58 2017 +0000

----------------------------------------------------------------------
 core/src/test/scala/unit/kafka/utils/JaasTestUtils.scala | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kafka/blob/2b6ba55c/core/src/test/scala/unit/kafka/utils/JaasTestUtils.scala
----------------------------------------------------------------------
diff --git a/core/src/test/scala/unit/kafka/utils/JaasTestUtils.scala b/core/src/test/scala/unit/kafka/utils/JaasTestUtils.scala
index a7f71b9..7055b7e 100644
--- a/core/src/test/scala/unit/kafka/utils/JaasTestUtils.scala
+++ b/core/src/test/scala/unit/kafka/utils/JaasTestUtils.scala
@@ -58,8 +58,7 @@ object JaasTestUtils {
 
   case class ScramLoginModule(username: String,
                               password: String,
-                              debug: Boolean = false,
-                              validUsers: Map[String, String] = Map.empty) {
+                              debug: Boolean = false) {
     def toJaasModule: JaasModule = {
       JaasModule(
         "org.apache.kafka.common.security.scram.ScramLoginModule",
@@ -67,7 +66,7 @@ object JaasTestUtils {
         entries = Map(
           "username" -> username,
           "password" -> password
-        ) ++ validUsers.map { case (user, pass) => s"user_$user" -> pass }
+        )
       )
     }
   }


Mime
View raw message