On Thu, 10 Sep 2026 18:00:51 GMT, Valerie Peng <[email protected]> wrote:

>> Andreas Chmielewski has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Add TLS 1.2 EMS algorithm alias in SunJCE
>
> src/jdk.crypto.cryptoki/share/classes/sun/security/pkcs11/P11TlsExtendedMasterSecretGenerator.java
>  line 134:
> 
>> 132:             throw new ProviderException(
>> 133:                     "Extended Master Secret session hash missing");
>> 134:         }
> 
> Move this check to `engineInit(...)` and throw 
> `InvalidAlgorithmParameterException` instead?

makes sense

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

PR Review Comment: https://git.openjdk.org/jdk/pull/32081#discussion_r3991280590

Reply via email to