[GitHub] [geode] albertogpz commented on issue #4204: GEODE-7157: SSLConfig&SSLConf..Factory thread-safe

["albertogpz (GitHub)" <gi...@apache.org>]

> I you went back to the initial condition (before this pr's work) wouldn't you be able to write a concurrent test that demonstrates that these classes are not thread safe? If so, then that test should start passing with this pr's changes.

The fact is that the method that had thread-safety issues (public static SSLConfig getSSLConfigForComponent(SecurableCommunicationChannel sslEnabledComponent)) has been eliminated.
Therefore, I could use in a test prior to the PR to see failures but I could not in the new version as it would not be present.


[ Full content available at: https://github.com/apache/geode/pull/4204 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org