diff options
author | Rob Crittenden <rcritten@redhat.com> | 2012-02-10 12:00:35 -0500 |
---|---|---|
committer | Martin Kosek <mkosek@redhat.com> | 2012-02-15 17:27:05 +0100 |
commit | f2da73e3671f38c00f625386a657e5260479d427 (patch) | |
tree | c3b8862d1ded9ca13216156ec4d82844deaec518 /ipaserver/install/ldapupdate.py | |
parent | af52c4d31e0740e0378035119f6463fc13d58737 (diff) | |
download | freeipa-f2da73e3671f38c00f625386a657e5260479d427.tar.gz freeipa-f2da73e3671f38c00f625386a657e5260479d427.tar.xz freeipa-f2da73e3671f38c00f625386a657e5260479d427.zip |
Correct update syntax in 30-s4u2proxy.update
Always have FQDN available in the update dictionary. There were cases
where it would contain the ldapi socket path and not the FQDN.
https://fedorahosted.org/freeipa/ticket/2147
Diffstat (limited to 'ipaserver/install/ldapupdate.py')
-rw-r--r-- | ipaserver/install/ldapupdate.py | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/ipaserver/install/ldapupdate.py b/ipaserver/install/ldapupdate.py index 8fbfeaf55..a2a94d31d 100644 --- a/ipaserver/install/ldapupdate.py +++ b/ipaserver/install/ldapupdate.py @@ -86,11 +86,11 @@ class LDAPUpdate: domain = ipautil.get_domain_name() libarch = self.__identify_arch() - if not self.ldapi: - fqdn = installutils.get_fqdn() - if fqdn is None: - raise RuntimeError("Unable to determine hostname") - else: + fqdn = installutils.get_fqdn() + if fqdn is None: + raise RuntimeError("Unable to determine hostname") + fqhn = fqdn # Save this for the sub_dict variable + if self.ldapi: fqdn = "ldapi://%%2fvar%%2frun%%2fslapd-%s.socket" % "-".join( self.realm.split(".") ) @@ -98,7 +98,7 @@ class LDAPUpdate: if not self.sub_dict.get("REALM") and self.realm is not None: self.sub_dict["REALM"] = self.realm if not self.sub_dict.get("FQDN"): - self.sub_dict["FQDN"] = fqdn + self.sub_dict["FQDN"] = fqhn if not self.sub_dict.get("DOMAIN"): self.sub_dict["DOMAIN"] = domain if not self.sub_dict.get("SUFFIX") and suffix is not None: |