[jdk8u-dev] Integrated: 8297955: LDAP CertStore should use LdapName and not String for DNs

[Andrew John Hughes]

On Fri, 24 Nov 2023 07:10:49 GMT, Andrew John Hughes <andrew at openjdk.org> wrote:

> [JDK-8308592](https://bugs.openjdk.org/browse/JDK-8308592): "Framework for CA interoperability testing" merges the Actalis test into a single test which handles multiple CAs. Before we can backport that, we thus need to fix the Actalis test by backporting [JDK-8297955](https://bugs.openjdk.org/browse/JDK-8297955) which also fixes [8224768](https://bugs.openjdk.org/browse/JDK-8224768)
> 
> The backport is mostly clean. The code changes apply cleanly once applied to `LDAPCertStore.java` rather than `LDAPCertStoreImpl.java` which doesn't exist in 8u (a refactoring in [JDK-8038084](https://bugs.openjdk.org/browse/JDK-8038084): "CertStore needs a way to add new CertStore types"). The unused `getName()` method needed to be removed as `name` is no longer a `String`. The `ProblemList.txt` change had to be manually added as did the copyright header change to `LDAPCertStore.java`.
> 
> The Actalis test passes after this backport. All other CA results match those on an unpatched build (GoogleCA currently seems to be broken on all 8u builds I had to hand)

This pull request has now been integrated.

Changeset: b2655a72
Author:    Andrew John Hughes <andrew at openjdk.org>
URL:       https://git.openjdk.org/jdk8u-dev/commit/b2655a72590ef27304c38638554bc25bda9793bc
Stats:     299 lines in 3 files changed: 56 ins; 88 del; 155 mod

8297955: LDAP CertStore should use LdapName and not String for DNs
8224768: Test ActalisCA.java fails

Reviewed-by: sgehwolf
Backport-of: 7f1047edba68cfe2fa2660030cb3dd1abad49e4f

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

PR: https://git.openjdk.org/jdk8u-dev/pull/388