Adapt src/test/ldap/t/001_auth.pl to work with openldap 2.5.

ldapsearch's deprecated -h/-p arguments were removed, need to use -H now -
which has been around for over 20 years.

As perltidy insists on reflowing the parameters anyway, change order and
"phrasing" to yield a less confusing layout (per suggestion from Tom Lane).

Discussion: https://postgr.es/m/20211009233850.wvr6apcrw2ai6cnj@alap3.anarazel.de
Backpatch: 11-, where the tests were added.

diff --git a/src/test/ldap/t/001_auth.pl b/src/test/ldap/t/001_auth.pl
index 3bc7672451ec88d8a39e42625079ab98b7b01c44..2c279a922b5ff78c239cbbb7509f9e8b64ea9598 100644 (file)
--- a/src/test/ldap/t/001_auth.pl
+++ b/src/test/ldap/t/001_auth.pl
@@ -127,10 +127,12 @@ while (1)
    last
      if (
        system_log(
-           "ldapsearch", "-h", $ldap_server, "-p",
-           $ldap_port,   "-s", "base",       "-b",
-           $ldap_basedn, "-D", $ldap_rootdn, "-y",
-           $ldap_pwfile, "-n", "'objectclass=*'") == 0);
+           "ldapsearch", "-sbase",
+           "-H",         $ldap_url,
+           "-b",         $ldap_basedn,
+           "-D",         $ldap_rootdn,
+           "-y",         $ldap_pwfile,
+           "-n",         "'objectclass=*'") == 0);
    die "cannot connect to slapd" if ++$retries >= 300;
    note "waiting for slapd to accept requests...";
    Time::HiRes::usleep(1000000);