On Fri, 20 Jan 2023 22:03:29 GMT, Hai-May Chao <hc...@openjdk.org> wrote:

>> Please review the fix to address the problem in keytool -genseckey and 
>> -importpass.
>
> Hai-May Chao has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Update with Max's comment

I said "one solution is to add RC2_40 and RC2_128" but I'm not sure if it's the 
right solution. If we resolve this issue in a separate issue, it means we 
believe the current decomposer implementation is correct and "RC2" SHOULD NOT 
cover "RC2_40". Thus. if one day we decide to disable AES, then we should 
disable all of AES_128, AES_192 and AES_256 since there are algorithm names 
like AES_192/OFB/NoPadding and PBEWithHmacSHA384AndAES_128. Let's find a 
agreement on this before closing out this issue.

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

PR: https://git.openjdk.org/jdk/pull/12056

Reply via email to