On Tue, 3 Dec 2024 19:56:09 GMT, Volkan Yazıcı <[email protected]> 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
