diff options
| author | Andres Freund <andres@anarazel.de> | 2021-10-19 10:14:49 -0700 |
|---|---|---|
| committer | Andres Freund <andres@anarazel.de> | 2021-10-19 11:15:45 -0700 |
| commit | 246a035f05c6accca34113de8e78f7afad90b329 (patch) | |
| tree | 5534aadd58ba267c73ee667f9d7072306c24ebed | |
| parent | 30e61a8cdc8624102ebdb1abbbb1e45457877c5c (diff) | |
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.
| -rw-r--r-- | src/test/ldap/t/001_auth.pl | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/src/test/ldap/t/001_auth.pl b/src/test/ldap/t/001_auth.pl index 3bc7672451e..2c279a922b5 100644 --- 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); |