RFR: 8358099: PEM spec updates [v2]

Sean Mullan mullan at openjdk.org
Wed Jun 4 20:44:53 UTC 2025


On Tue, 3 Jun 2025 16:24:44 GMT, Sean Mullan <mullan at openjdk.org> wrote:

>> Anthony Scarpino has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   comments
>
> src/java.base/share/classes/java/security/PEMDecoder.java line 127:
> 
>> 125:  * @implNote An implementation may support other PEM types and
>> 126:  * {@code DEREncodable} objects. This implementation additionally supports
>> 127:  * PEM types:  {@code X509 CERTIFICATE}, {@code X.509 CERTIFICATE}, {@code CRL},
> 
> Suggest to insert "the following" before "PEM types"

"the following" should not be in the first sentence, only the second sentence, as:

"This implementation additionally supports the following PEM types: ..."

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25620#discussion_r2127402367


More information about the security-dev mailing list