tests: Add LDAP URI to ldappasswd explicitly
Tests should always rely on api.env.* values when possible.
Without this running the tests remotely can result in errors such
as ldap{search,modify,passwd} attempting to connect to the
wrong URI and failing.
https://fedorahosted.org/freeipa/ticket/6622
Reviewed-By: Alexander Bokovoy <abokovoy@redhat.com>
Reviewed-By: Rob Crittenden <rcritten@redhat.com>
This commit is contained in:
1 parent
7a3da27816
commit
53abf0105b
2 files changed
+3
-2
No files matched your search
@@ -1326,9 +1326,10 @@ 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)
|
||||
|
||||
args = [paths.LDAPPASSWD, '-D', userdn, '-w', oldpw, '-a', oldpw,
|
||||
'-s', newpw, '-x']
|
||||
'-s', newpw, '-x', '-H', master_ldap_uri]
|
||||
master.run_command(args)
|
||||
|
||||
|
||||
|
||||
+1
-1
@@ -750,7 +750,7 @@ def unlock_principal_password(user, oldpw, newpw):
|
||||
user, api.env.container_user, api.env.basedn)
|
||||
|
||||
args = [paths.LDAPPASSWD, '-D', userdn, '-w', oldpw, '-a', oldpw,
|
||||
'-s', newpw, '-x']
|
||||
'-s', newpw, '-x', '-H', api.env.ldap_uri]
|
||||
return run(args)
|
||||
|
||||
|
||||
|
||||
Reference in new issue
Block a user