From 613a485c4798b38a65042e9d78968896c590fdd0 Mon Sep 17 00:00:00 2001 From: Michael DeHaan Date: Mon, 25 Feb 2008 16:53:08 -0500 Subject: Certmaster hostname check is different than minion check --- certmaster/certmaster.py | 2 +- certmaster/utils.py | 32 ++++++++++++++++---------------- 2 files changed, 17 insertions(+), 17 deletions(-) diff --git a/certmaster/certmaster.py b/certmaster/certmaster.py index fe5dcbc..9ebd66b 100755 --- a/certmaster/certmaster.py +++ b/certmaster/certmaster.py @@ -39,7 +39,7 @@ class CertMaster(object): def __init__(self, conf_file=CERTMASTER_CONFIG): self.cfg = read_config(conf_file, CMConfig) - usename = utils.get_hostname() + usename = utils.get_hostname(talk_to_certmaster=False) mycn = '%s-CA-KEY' % usename self.ca_key_file = '%s/funcmaster.key' % self.cfg.cadir diff --git a/certmaster/utils.py b/certmaster/utils.py index 7927c55..3dec6a5 100755 --- a/certmaster/utils.py +++ b/certmaster/utils.py @@ -75,7 +75,7 @@ def is_error(result): return True return False -def get_hostname(): +def get_hostname(talk_to_certmaster=True): """ "localhost" is a lame hostname to use for a key, so try to get a more meaningful hostname. We do this by connecting to the certmaster @@ -94,23 +94,23 @@ def get_hostname(): if ip != "127.0.0.1": return hostname + if talk_to_certmaster: + config_file = '/etc/certmaster/minion.conf' + config = read_config(config_file, MinionConfig) - config_file = '/etc/certmaster/minion.conf' - config = read_config(config_file, MinionConfig) - - server = config.certmaster - port = 51235 + server = config.certmaster + port = 51235 - try: - s = socket.socket() - s.settimeout(5) - s.connect((server, port)) - (intf, port) = s.getsockname() - hostname = socket.gethostbyaddr(intf)[0] - s.close() - except: - s.close() - raise + try: + s = socket.socket() + s.settimeout(5) + s.connect((server, port)) + (intf, port) = s.getsockname() + hostname = socket.gethostbyaddr(intf)[0] + s.close() + except: + s.close() + raise return hostname -- cgit