jolshan commented on code in PR #14721:
URL: https://github.com/apache/kafka/pull/14721#discussion_r1388381937
##########
clients/src/main/java/org/apache/kafka/common/security/ssl/SslFactory.java:
##########
@@ -99,7 +100,7 @@ public void configure(Map<String, ?> configs) throws
KafkaException {
try {
SslEngineValidator.validate(builder, builder);
} catch (Exception e) {
- throw new ConfigException("A client SSLEngine created with the
provided settings " +
+ throw new InvalidConfigurationException("A client SSLEngine
created with the provided settings " +
Review Comment:
Usages here are a bit tricky to track. So many overrided "configure" methods
that go in circles 😅
@bob-barrett do you have a simplified flow for this to confirm it won't
cause compatibility issues?
##########
clients/src/main/java/org/apache/kafka/common/security/ssl/SslFactory.java:
##########
@@ -99,7 +100,7 @@ public void configure(Map<String, ?> configs) throws
KafkaException {
try {
SslEngineValidator.validate(builder, builder);
} catch (Exception e) {
- throw new ConfigException("A client SSLEngine created with the
provided settings " +
+ throw new InvalidConfigurationException("A client SSLEngine
created with the provided settings " +
Review Comment:
Usages here are a bit tricky to track. So many overrided "configure" methods
that go in circles 😅
@bob-barrett do you have a simplified flow for this to confirm it won't
cause compatibility issues?
--
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]
