This is an automated email from the git hooks/post-receive script.
mhonek pushed a commit to branch 389-ds-base-1.3.9
in repository 389-ds-base.
The following commit(s) were added to refs/heads/389-ds-base-1.3.9 by this push:
new 89a7f26 Ticket 50138 - db2bak.pl -P LDAPS does not work when nsslapd-securePort
is missing
89a7f26 is described below
commit 89a7f266822ee7821a0c6208ed3b8e7ee019e7eb
Author: Matúš Honěk <mhonek(a)redhat.com>
AuthorDate: Wed Jan 9 13:49:25 2019 +0100
Ticket 50138 - db2bak.pl -P LDAPS does not work when nsslapd-securePort is missing
Bug Description:
When nsslapd-securePort is not set in dse.ldif, DSUtil (in case of LDAPS)
interpolates ldapmodify (and other) call to a URI ending only with a colon
because obtained securePort is just an empty string. The very same turns out to
happen for the nsslapd-port.
Fix Description:
Hardcode the default ports, which is a short variant of omitting the
colon and port, and which would be chosen by the OpenLDAP utils any way.
https://pagure.io/389-ds-base/issue/50138
Author: mhonek
Review by: mreynolds (thanks!)
---
ldap/admin/src/scripts/DSUtil.pm.in | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/ldap/admin/src/scripts/DSUtil.pm.in b/ldap/admin/src/scripts/DSUtil.pm.in
index 791464d..c2a9af0 100644
--- a/ldap/admin/src/scripts/DSUtil.pm.in
+++ b/ldap/admin/src/scripts/DSUtil.pm.in
@@ -1239,7 +1239,7 @@ sub get_info {
$info{host} = $entry->getValues("nsslapd-localhost");
}
if($info{port} eq ""){
- $info{port} = $entry->getValues("nsslapd-port");
+ $info{port} = $entry->getValues("nsslapd-port") || "389";
}
if($info{rootdn} eq ""){
$info{rootdn} = $entry->getValues("nsslapd-rootdn");
@@ -1253,7 +1253,7 @@ sub get_info {
$ENV{LDAPTLS_CACERTDIR}=$info{certdir};
}
$info{security} = $entry->getValues("nsslapd-security");
- $info{secure_port} = $entry->getValues("nsslapd-securePort");
+ $info{secure_port} = $entry->getValues("nsslapd-securePort") ||
"636";
$info{ldapi} = $entry->getValues("nsslapd-ldapilisten");
$info{autobind} = $entry->getValues("nsslapd-ldapiautobind");
$value = $entry->getValues("nsslapd-ldapifilepath");
--
To stop receiving notification emails like this one, please contact
the administrator of this repository.