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