On Wed, 20 Nov 2024 14:44:38 GMT, Sean Mullan <[email protected]> wrote:
>> Artur Barashev has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Remove the empty lines added
>
> test/jdk/javax/net/ssl/DTLS/DTLSMFLNTest.java line 56:
>
>> 54: public class DTLSMFLNTest {
>> 55: public static void main(String[] args) {
>> 56: SecurityUtils.removeFromDisabledTlsAlgs("TLS_RSA_*");
>
> Are you sure this is needed? This test passes for me if I comment out this
> line.
This is strange, I remember this test failing for sure when running locally,
but now it passes for me as well. It runs bunch of test cases actually and it
reported something like `38 out of 72 tests failed`. Must be something else was
changed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22163#discussion_r1850521392
