On Thu, 21 Nov 2024 09:44:50 GMT, Volkan Yazıcı <d...@openjdk.org> wrote:
>> Removes `SecurityManager` et al. from `SocksSocketImpl`. `tier2` and `tier3` >> tests have passed – CI run links are available in the ticket. > > Volkan Yazıcı has updated the pull request incrementally with one additional > commit since the last revision: > > Revert to using a synchronized method to perform the connection LGTM if you implement @liach's suggestion. ------------- PR Review: https://git.openjdk.org/jdk/pull/22271#pullrequestreview-2452012370
