On Tue, 30 Jun 2026 17:08:03 GMT, Andreas Chmielewski 
<[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).
>
> Andreas Chmielewski has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Review comments

test/jdk/javax/net/ssl/ciphersuites/BulkCipherDisabledAlgorithms.java line 212:

> 210:     }
> 211: 
> 212:     private static class SSLServer extends SSLContextTemplate implements 
> Runnable, AutoCloseable {

Did you consider `BulkCipherDisabledAlgorithms` extending `SSLSocketTemplate` 
instead of implementing  Server/Client functionality yourself? That would 
simplify the test considerably.

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

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

Reply via email to