RFR: 8344219: Remove calls to SecurityManager and doPrivileged in java.net.SocksSocketImpl after JEP 486 integration [v3]
Daniel Fuchs
dfuchs at openjdk.org
Thu Nov 21 16:27:16 UTC 2024
On Thu, 21 Nov 2024 09:44:50 GMT, Volkan Yazıcı <duke at 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
More information about the net-dev
mailing list