On Wed, 21 Dec 2022 20:07:40 GMT, Xue-Lei Andrew Fan <xue...@openjdk.org> wrote:
>> Volker Simonis has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Optimized initialisation of currentKeyID and deletion of expired session
>> keys as proposed by @schlosna
>
> src/java.base/share/classes/sun/security/ssl/SSLSessionContextImpl.java line
> 79:
>
>> 77: private final Map<Integer, // Maps session keys to session
>> state
>> 78: SessionTicketExtension.StatelessKey> keyHashMap = new
>> ConcurrentHashMap<>();
>> 79:
>
> It may be not necessary to allocate the map for client session context. It
> may be fine to me the map allocation to constructor.
>
>
> private final Map<Integer, SessionTicketExtension.StatelessKey> keyHashMap;
> SSLSessionContextImpl(boolean server) {
> ....
> if (server) {
> keyHashMap = ...
> currentKeyID = ...
> } else {
> keyHashMap = empty map
> }
> }
Changed.
-------------
PR: https://git.openjdk.org/jdk/pull/11590
