From 20d8286b60b6f3f597912d16afd04ef29ad248c8 Mon Sep 17 00:00:00 2001 From: Sergey Orlov Date: Tue, 22 Jan 2019 15:24:05 +0100 Subject: [PATCH] ipatests: fix ldap server url master.external_hostname was used to construct ldap url which caused ldappasswd utility to exit with error due to host name mismatch in client certificate. master.hostname should be used instead as this name is used to generate certificate. Fixes https://pagure.io/freeipa/issue/7844 Signed-off-by: Sergey Orlov Reviewed-By: Florence Blanc-Renaud Reviewed-By: Rob Crittenden --- ipatests/pytest_ipa/integration/tasks.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ipatests/pytest_ipa/integration/tasks.py b/ipatests/pytest_ipa/integration/tasks.py index 4b5ddd855..72cef880e 100644 --- a/ipatests/pytest_ipa/integration/tasks.py +++ b/ipatests/pytest_ipa/integration/tasks.py @@ -1488,7 +1488,7 @@ def ldappasswd_user_change(user, oldpw, newpw, master): basedn = master.domain.basedn userdn = "uid={},{},{}".format(user, container_user, basedn) - master_ldap_uri = "ldap://{}".format(master.external_hostname) + master_ldap_uri = "ldap://{}".format(master.hostname) args = [paths.LDAPPASSWD, '-D', userdn, '-w', oldpw, '-a', oldpw, '-s', newpw, '-x', '-ZZ', '-H', master_ldap_uri] @@ -1500,7 +1500,7 @@ def ldappasswd_sysaccount_change(user, oldpw, newpw, master): basedn = master.domain.basedn userdn = "uid={},{},{}".format(user, container_sysaccounts, basedn) - master_ldap_uri = "ldap://{}".format(master.external_hostname) + master_ldap_uri = "ldap://{}".format(master.hostname) args = [paths.LDAPPASSWD, '-D', userdn, '-w', oldpw, '-a', oldpw, '-s', newpw, '-x', '-ZZ', '-H', master_ldap_uri]