Actually run the renew hook

This commit is contained in:
Peter Eckersley 2016-03-29 17:58:53 -07:00
parent 5943dc7c3d
commit ad5a08f5fc
3 changed files with 3 additions and 1 deletions

View file

@ -67,7 +67,7 @@ def _run_hook(shell_cmd):
:returns: stderr if there was any"""
cmd = Popen(shell_cmd, shell=True, stdout=PIPE, stderr=PIPE)
cmd = Popen(shell_cmd, shell=True, stdout=PIPE, stderr=PIPE, stdin=PIPE)
_out, err = cmd.communicate()
if cmd.returncode != 0:
logger.error('Hook command "%s" returned error code %d', shell_cmd, cmd.returncode)

View file

@ -243,6 +243,7 @@ def renew_cert(config, domains, le_client, lineage):
renewal_conf = configuration.RenewerConfiguration(config.namespace)
lineage.save_successor(prior_version, new_cert, new_key.pem, new_chain, renewal_conf)
lineage.update_all_links_to(lineage.latest_common_version())
hooks.renew_hook(config, domains, lineage.live_dir)
# TODO: Check return value of save_successor

View file

@ -252,6 +252,7 @@ class RenewableCert(object): # pylint: disable=too-many-instance-attributes
self.privkey = self.configuration["privkey"]
self.chain = self.configuration["chain"]
self.fullchain = self.configuration["fullchain"]
self.live_dir = os.path.dirname(self.cert)
self._fix_symlinks()
self._check_symlinks()