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