RFR: 8314323: Implement JEP 527: TLS 1.3 Hybrid Key Exchange [v3]

Artur Barashev abarashev at openjdk.org
Tue Oct 14 18:18:51 UTC 2025


On Sun, 5 Oct 2025 12:21:02 GMT, Hai-May Chao <hchao at openjdk.org> wrote:

>> src/java.base/share/classes/sun/security/ssl/NamedGroup.java line 316:
>> 
>>> 314:                 // Skip AlgorithmParameters for KEMs (not supported)
>>> 315:                 if (namedGroupSpec == NamedGroupSpec.NAMED_GROUP_KEM) {
>>> 316:                     if (defaultProviderName == null) {
>> 
>> We assume that if provider is not null then it must be DH without doing any checks to confirm that. It would be cleaner to call getProvider() instead.
>> 
>>> Provider p = getProvider();
>>                     if (p == null) {
>>                         KeyFactory.getInstance(name);
>>                     } else {
>>                         KeyFactory.getInstance(name, p);
>>                     }
>
> Done.

It's actually unchanged, do you plan to do it in the next iteration?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27614#discussion_r2430049480


More information about the security-dev mailing list