RFR: JDK-8285263 Minor cleanup could be done in java.security [v5]

Mark Powers duke at openjdk.java.net
Wed Jun 8 22:29:39 UTC 2022


On Wed, 8 Jun 2022 16:12:05 GMT, Weijun Wang <weijun at openjdk.org> wrote:

>> Mark Powers has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 11 commits:
>> 
>>  - Merge
>>  - fourth iteration
>>  - Merge
>>  - Merge
>>  - third iteration
>>  - Merge
>>  - Merge
>>  - second iteration
>>  - Merge
>>  - Merge
>>  - ... and 1 more: https://git.openjdk.java.net/jdk/compare/4d6fb515...44140a01
>
> src/java.base/share/classes/java/security/cert/LDAPCertStoreParameters.java line 34:
> 
>> 32:  * name and port number) to implementations of the LDAP {@code CertStore}
>> 33:  * algorithm. However, if you are retrieving certificates or CRLs from
>> 34:  * a ldap URI as specified by RFC 5280, use the
> 
> Isn't `ldap` pronounced `L-Dap`?

You are right. IntelliJ was wrong and I didn't catch it.

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

PR: https://git.openjdk.java.net/jdk/pull/8319



More information about the security-dev mailing list