Skip to content

Commit 9393507

Browse files
committed
fix
1 parent d151ba5 commit 9393507

File tree

7 files changed

+8
-6
lines changed

7 files changed

+8
-6
lines changed

clients/src/main/java/org/apache/kafka/common/security/JaasUtils.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -37,11 +37,11 @@ public final class JaasUtils {
3737
private JaasUtils() {
3838
}
3939

40-
public static void allowDefaultJaasAndCustomJass(String... customJaas) {
40+
public static void allowDefaultJaasAndCustomJaas(String... customJaas) {
4141
List<String> jaasModules = new ArrayList<>();
4242
jaasModules.add(ALLOWED_LOGIN_MODULES_DEFAULT);
4343
jaasModules.addAll(Arrays.asList(customJaas));
44-
System.setProperty(org.apache.kafka.common.security.JaasUtils.ALLOWED_LOGIN_MODULES_CONFIG, String.join(",", jaasModules));
44+
System.setProperty(ALLOWED_LOGIN_MODULES_CONFIG, String.join(",", jaasModules));
4545
}
4646

4747
}

clients/src/test/java/org/apache/kafka/common/security/authenticator/ClientAuthenticationFailureTest.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ public void setup() throws Exception {
7676
TestJaasConfig testJaasConfig = TestJaasConfig.createConfiguration("PLAIN", Collections.singletonList("PLAIN"));
7777
testJaasConfig.setClientOptions("PLAIN", TestJaasConfig.USERNAME, "anotherpassword");
7878
server = createEchoServer(securityProtocol);
79-
JaasUtils.allowDefaultJaasAndCustomJass();
79+
JaasUtils.allowDefaultJaasAndCustomJaas();
8080
}
8181

8282
@AfterEach

clients/src/test/java/org/apache/kafka/common/security/authenticator/LoginManagerTest.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ public void setUp() {
6060
" required user=\"digestuser\" password=\"digest-secret\";");
6161
TestJaasConfig.createConfiguration("SCRAM-SHA-256",
6262
Collections.singletonList("SCRAM-SHA-256"));
63-
JaasUtils.allowDefaultJaasAndCustomJass("org.apache.kafka.common.security.authenticator.TestDigestLoginModule");
63+
JaasUtils.allowDefaultJaasAndCustomJaas("org.apache.kafka.common.security.authenticator.TestDigestLoginModule");
6464
}
6565

6666
@AfterEach

clients/src/test/java/org/apache/kafka/common/security/authenticator/SaslAuthenticatorFailureDelayTest.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ public void setup() throws Exception {
7575
credentialCache = new CredentialCache();
7676
SaslAuthenticatorTest.TestLogin.loginCount.set(0);
7777
startTimeMs = time.milliseconds();
78-
JaasUtils.allowDefaultJaasAndCustomJass();
78+
JaasUtils.allowDefaultJaasAndCustomJaas();
7979
}
8080

8181
@AfterEach

clients/src/test/java/org/apache/kafka/common/security/authenticator/SaslAuthenticatorTest.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -175,7 +175,7 @@ public void setup() throws Exception {
175175
saslClientConfigs = clientCertStores.getTrustingConfig(serverCertStores);
176176
credentialCache = new CredentialCache();
177177
TestLogin.loginCount.set(0);
178-
JaasUtils.allowDefaultJaasAndCustomJass("org.apache.kafka.common.security.authenticator.TestDigestLoginModule",
178+
JaasUtils.allowDefaultJaasAndCustomJaas("org.apache.kafka.common.security.authenticator.TestDigestLoginModule",
179179
"org.apache.kafka.common.security.authenticator.SaslAuthenticatorTest$TestPlainLoginModule");
180180
}
181181

core/src/test/scala/integration/kafka/api/SaslEndToEndAuthorizationTest.scala

+1
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ abstract class SaslEndToEndAuthorizationTest extends EndToEndAuthorizationTest {
4646
producerConfig.put(SaslConfigs.SASL_JAAS_CONFIG, clientLoginContext)
4747
consumerConfig.put(SaslConfigs.SASL_JAAS_CONFIG, clientLoginContext)
4848
adminClientConfig.put(SaslConfigs.SASL_JAAS_CONFIG, clientLoginContext)
49+
4950
val superuserLoginContext = jaasAdminLoginModule(kafkaClientSaslMechanism)
5051
superuserClientConfig.put(SaslConfigs.SASL_JAAS_CONFIG, superuserLoginContext)
5152
super.setUp(testInfo)

core/src/test/scala/integration/kafka/api/SaslSslAdminIntegrationTest.scala

+1
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,7 @@ class SaslSslAdminIntegrationTest extends BaseAdminIntegrationTest with SaslSetu
7070
this.serverConfig.setProperty(DelegationTokenManagerConfigs.DELEGATION_TOKEN_SECRET_KEY_CONFIG, secretKey)
7171
this.serverConfig.setProperty(DelegationTokenManagerConfigs.DELEGATION_TOKEN_EXPIRY_TIME_MS_CONFIG, Long.MaxValue.toString)
7272
this.serverConfig.setProperty(DelegationTokenManagerConfigs.DELEGATION_TOKEN_MAX_LIFETIME_CONFIG, Long.MaxValue.toString)
73+
7374
setUpSasl()
7475
super.setUp(testInfo)
7576
setInitialAcls()

0 commit comments

Comments
 (0)