[GitHub] [pulsar] Jason918 commented on a change in pull request #14253: Check ``getTlsTrustStorePath`` NPE when user forget to set it.

[GitBox <gi...@apache.org>]

Jason918 commented on a change in pull request #14253:
URL: https://github.com/apache/pulsar/pull/14253#discussion_r805110307



##########
File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/PulsarChannelInitializer.java
##########
@@ -72,7 +72,10 @@ public PulsarChannelInitializer(ClientConfigurationData conf, Supplier<ClientCnx
         if (tlsEnabled) {
             if (tlsEnabledWithKeyStore) {
                 AuthenticationDataProvider authData1 = conf.getAuthentication().getAuthData();
-
+                if (conf.getTlsTrustStorePath() == null) {
+                    throw new RuntimeException("Failed to create TLS context, Due to empty "

Review comment:
       ```suggestion
                       throw new RuntimeException("Failed to create TLS context, due to empty "
   ```




-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org