RFR: 8298420: PEM API: Implementation (Preview) [v9]

Anthony Scarpino ascarpino at openjdk.org
Thu Oct 31 00:11:33 UTC 2024


On Wed, 30 Oct 2024 20:14:04 GMT, Weijun Wang <weijun at openjdk.org> wrote:

>> Anthony Scarpino has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   apparently <p> can't be before a @implNote.. Who know.
>
> src/java.base/share/classes/javax/crypto/EncryptedPrivateKeyInfo.java line 339:
> 
>> 337:      * @throws IllegalArgumentException when an argument causes an
>> 338:      * initialization error.
>> 339:      * @throws SecurityException on a cryptographic errors.
> 
> Oh, I didn't notice this before. Have we decided to repurpose `SecurityException` for this usage now that there will be no more Security Manager?

I'm not sure if it was exclusively a Security Manager exception, but @seanjmullan had said previously that `SecurityException` was used elsewhere.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17543#discussion_r1823616277


More information about the security-dev mailing list