On Thu, 25 Jun 2026 15:23:59 GMT, Artur Barashev <[email protected]> wrote:

>> These mappings were previously missing, causing
>> jdk.tls.disabledAlgorithms constraints using component names (e.g.
>>         "AES_128_GCM", "AES_256_GCM", "CHACHA20_POLY1305") to not
>> consistently match the corresponding TLS cipher suites.
>> 
>> 
>> 
>> 
>> ---------
>> - [ x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> test/jdk/sun/security/ssl/CipherSuite/TLS13DisabledAlgorithm.java line 29:
> 
>> 27:  * @modules jdk.crypto.ec
>> 28:  * @library /test/lib /javax/net/ssl/templates
>> 29:  * @run main/othervm -enablesystemassertions TLS13DisabledAlgorithm
> 
> Why we need `-enablesystemassertions` flag?

Also, I don't think we need `othervm` for this test. Basically this whole line 
can be removed since we run `main` by default anyhow.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/31633#discussion_r3476017156

Reply via email to