RFR: 8293345: SunPKCS11 provider checks on PKCS11 Mechanism are problematic [v2]

[Mark Powers]

On Mon, 8 Apr 2024 19:33:25 GMT, Valerie Peng <valeriep at openjdk.org> wrote:

>> Existing legacy mechanism check disables mechanism(s) when the support is partial, e.g. supports decryption but not encryption, or supports verification but not signing. Some mechanisms can be used for both encryption/decryption and sign/verify such as RSA related ones. If the particular mechanism supports sign/verify/decryption but not encryption, it'd be disabled as a result. Fine tune the legacy mechanism check with the service type, i.e. supports encryption for Cipher,  sign for Signature, so the mechanism is disabled based on the service type.
>> For completeness sake, I also added a PKCS11 provider configuration option to control this.  If not set, SunPKCS11 provider will disable legacy mechanisms by default.
>
> Valerie Peng has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Update to match CSR for disableLegacy -> allowLegacy name change

Your changes look fine to me.

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

PR Comment: https://git.openjdk.org/jdk/pull/18387#issuecomment-2103464943