jsinovassin opened a new pull request, #734: URL: https://github.com/apache/unomi/pull/734
This pull request refactors the way SSL context is handled when creating the Elasticsearch client. The main changes focus on ensuring that the SSL context is only set when the "trust all certificates" option is enabled. **Elasticsearch client initialization improvements:** * Refactored the client builder logic in `ElasticSearchPersistenceServiceImpl.java` to initialize `ElasticsearchClientFactory.ClientBuilder` once and only set the SSL context if `sslTrustAllCertificates` is true, avoiding unnecessary SSL context usage. * Moved the call to set the SSL context (`sslContext`) on the client builder inside the conditional block for `sslTrustAllCertificates`, ensuring it is only set when appropriate. -- 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]
