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

Andrew John Hughes andrew at openjdk.org
Tue Nov 28 03:25:32 UTC 2023


> [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)

Andrew John Hughes has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase.

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

Changes:
  - all: https://git.openjdk.org/jdk8u-dev/pull/388/files
  - new: https://git.openjdk.org/jdk8u-dev/pull/388/files/9fea5694..9fea5694

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=388&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=388&range=00-01

  Stats: 0 lines in 0 files changed: 0 ins; 0 del; 0 mod
  Patch: https://git.openjdk.org/jdk8u-dev/pull/388.diff
  Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/388/head:pull/388

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


More information about the jdk8u-dev mailing list