nandorsoma commented on PR #7668:
URL: https://github.com/apache/nifi/pull/7668#issuecomment-1707320018

   > Thanks for the improvement @nandorsoma.
   > 
   > Unfortunately, the design of the Splunk SDK `Service` class is not suited 
for a configurable `SSLSocketFactory` based on the static configuration 
approach. Calling the `Service.setSSLSocketFactory` method changes the Socket 
Factory for all instances of the Splunk Service, which would impact all 
instances of the Processor, and other Processors that use the Splunk SDK.
   > 
   > One option might be to use instance class-loading, as used in several 
Hadoop-based Processors. Another more involved option would be the replace the 
Splunk SDK client with custom network client libraries that support 
configurable TLS properties.
   
   Thanks for the quick review @exceptionfactory! You are right about the issue 
regarding static configuration. Thank you for bringing that to my attention!


-- 
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]

Reply via email to