frankvicky commented on code in PR #16724:
URL: https://github.com/apache/kafka/pull/16724#discussion_r1697183779
##########
clients/src/test/java/org/apache/kafka/common/security/authenticator/LoginManagerTest.java:
##########
@@ -153,6 +163,32 @@ public void testLoginManagerWithDifferentConfigs() throws
Exception {
verifyLoginManagerRelease(staticLogin2, 2, staticContext, configs2);
}
+ @Test
+ public void testLoginException() throws Exception {
+ Map<String, Object> config = new HashMap<>();
+ config.put("sasl.jaas.config", dynamicPlainContext);
Review Comment:
I think we could use `SaslConfigs.SASL_JAAS_CONFIG` here instead of this
plain string literal.
I have seen the following config key also use the const from `SaslConfigs`,
so it would be nice to align with them.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
