intrigus-lgtm opened a new pull request #2304: URL: https://github.com/apache/calcite/pull/2304
Removes the insecure, unused `TrustAllSslSocketFactory` class. Following history, this leads us to https://github.com/apache/calcite/commit/33013ac160c9aa5e5a2b4373976fc5109f622197#diff-1f23cb4c9c91d3536d23f20dfe702e6d0f4068418283c573cadad05ab32f4a3b and https://github.com/julianhyde/optiq-splunk/commit/262688f1bf65f4e884dc3496e13c6bb992ff5701#diff-609118c1f9bfb010bcd846c0a85b9c637190a4ccd83d19bcde427c7ff6a81d11 The only user was `HttpUtils` https://github.com/apache/calcite/commit/33013ac160c9aa5e5a2b4373976fc5109f622197#diff-6c6ed01845b745f36f3d925b8aab55d18e12d6c47c7261793383e3dde370fdd5R46 The insecure code in `HttpUtils` has been removed in [CALCITE-4298] ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
