RFR: 8344229: Revisit SecurityManager usage in jdk.httpserver after JEP 486 integration
Daniel Fuchs
dfuchs at openjdk.org
Tue Dec 3 17:26:42 UTC 2024
On Tue, 3 Dec 2024 10:44:37 GMT, Volkan Yazıcı <duke at openjdk.org> wrote:
> Removes `SecurityManager` and friends from `jdk.httpserver`. Will share `tier1..2` results soon.
>
> **Methodology:** Case-insensitively and recursively searched for the occurrences of `("removal"|secu|privi)` in `httpserver`-named folders.
Good cleanup!
src/jdk.httpserver/share/classes/sun/net/httpserver/ServerConfig.java line 53:
> 51:
> 52: private static long idleTimerScheduleMillis;
> 53: private static long idleIntervalMillis;
while you're at it you could make these two fields final as well and use temporary variables in the static block to make sure they're assigned only once.
-------------
PR Review: https://git.openjdk.org/jdk/pull/22510#pullrequestreview-2476336083
PR Review Comment: https://git.openjdk.org/jdk/pull/22510#discussion_r1868117269
More information about the net-dev
mailing list