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

[Andrew John Hughes]

On Fri, 24 Nov 2023 14:46:15 GMT, Severin Gehwolf <sgehwolf at openjdk.org> wrote:

> This looks OK. `getName()` got removed in later JDKs, by JDK-8224729. This patch is fine, though. No need to drag in that additional dependency.

Thanks. I'd assumed it was part of the big refactoring in [JDK-8038084](https://bugs.openjdk.org/browse/JDK-8038084) - and it's definitely a no for that one - but it seems it survived a bit longer and 8224729 was also backported for the 11 backport of this. I agree just removing `getName` is enough and we don't need to add more dependencies with that change.

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

PR Comment: https://git.openjdk.org/jdk8u-dev/pull/388#issuecomment-1825809567