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