RFR: 8344229: Revisit SecurityManager usage in jdk.httpserver after JEP 486 integration [v2]

Daniel Fuchs dfuchs at openjdk.org
Tue Dec 3 20:05:47 UTC 2024


On Tue, 3 Dec 2024 19:56:09 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.
>
> Volkan Yazıcı has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Finalize constants in `ServerConfig`

src/jdk.httpserver/share/classes/sun/net/httpserver/ServerConfig.java line 80:

> 78:         idleIntervalMillis = Math.multiplyExact(
> 79:                 providedIdleIntervalSecs > 0 ? providedIdleIntervalSecs : DEFAULT_IDLE_INTERVAL_IN_SECS,
> 80:                 1000);

That's a change of behaviour - maybe we should keep the original behaviour that was not unreasonable: if the result of the computation overflows, use the implementation default value.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22510#discussion_r1868295934


More information about the net-dev mailing list