RFR: 8344185: Remove calls to SecurityManager in sun.net.ftp

Michael McMahon michaelm at openjdk.org
Thu Nov 14 18:00:32 UTC 2024


On Thu, 14 Nov 2024 12:57:25 GMT, Eirik Bjørsnøs <eirbjo at openjdk.org> wrote:

> Please review this PR which removes uses of `AccessController::doPrivileged` in `sun/net/ftp`.
> 
> Changes are mostly straightforward, one exception is the removal of the somewhat exotic unwrapping of temporary arrays in the static initializer of `impl/FtpClient`.
> 
> Testing: `test/jdk/sun/net/ftp` run green. GHA results pending.

Looks fine to me as well. I checked the FTP specific tests and they pass. But, would be good to run all of tier2.

-------------

Marked as reviewed by michaelm (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/22105#pullrequestreview-2436826838


More information about the net-dev mailing list