RFR: 8328608: Multiple NewSessionTicket support for TLS [v4]

Anthony Scarpino ascarpino at openjdk.org
Tue Aug 20 17:40:08 UTC 2024


On Mon, 19 Aug 2024 19:26:19 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:

>> Anthony Scarpino has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - revert to synchronized
>>  - code review changes
>
> src/java.base/share/classes/sun/security/util/Cache.java line 478:
> 
>> 476:             removeImpl(key);
>> 477:             if (DEBUG) {
>> 478:                 System.out.println("Ignoring expired entry: ");
> 
> did you intend to print the entry contents here?

I'm not sure.  It doesn't seem like I would be printing a list of entries here.  I reverted the code.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19465#discussion_r1723702407



More information about the security-dev mailing list