RFR: 8328608: Multiple NewSessionTicket support for TLS [v3]
Anthony Scarpino
ascarpino at openjdk.org
Thu Aug 1 19:38:36 UTC 2024
On Mon, 29 Jul 2024 17:18:59 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:
>> Anthony Scarpino has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 21 commits:
>>
>> - Rework TLSBase for better testing
>> - Tests working
>> - Merge branch 'master' into nst-multi
>> - new changes
>> - remove frag issue
>> - Comments, remove thread, set NST default to 1, allow 0
>> - comment cleanup
>> - Merge branch 'master' into nst-multi
>> - copyright & cleanup
>> - oops BAOS
>> - ... and 11 more: https://git.openjdk.org/jdk/compare/3796fdfc...35bfe799
>
> test/jdk/sun/security/ssl/SSLSessionImpl/MultiNSTParallel.java line 199:
>
>> 197: System.err.println("released: " + t.getName());
>> 198: }
>> 199: } catch (Exception e) {
>
> is this catch necessary?
Apparently not.. must have been an old catch
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19465#discussion_r1700724466
More information about the security-dev
mailing list