On Tue, 25 Jun 2024 09:14:12 GMT, Daniel Jeliński <djelin...@openjdk.org> wrote:
>> Anthony Scarpino has updated the pull request incrementally with three
>> additional commits since the last revision:
>>
>> - remove frag issue
>> - Comments, remove thread, set NST default to 1, allow 0
>> - comment cleanup
>
> src/java.base/share/classes/sun/security/ssl/SSLSessionContextImpl.java line
> 93:
>
>> 91: if (server) {
>> 92: sessionCache = Cache.newSoftMemoryCache(cacheLimit, timeout);
>> 93: sessionHostPortCache = Cache.newSoftMemoryCache(cacheLimit,
>> timeout);
>
> preexisting, but I wonder if we could use nullCache here and in client's
> sessionCache
Let's deal with that outside of this RFE, but there are tests that use it and
I'm pretty sure this has public API usage with
SSLSessionContext.getSession(byte[])
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19465#discussion_r1657634183
