Updated todos

This commit is contained in:
Brad Warren
2015-10-19 19:13:48 -07:00
parent 3885a40c42
commit a643289662
3 changed files with 3 additions and 8 deletions
+1 -2
View File
@@ -95,7 +95,7 @@ def renew(cert, old_version):
sans = crypto_util.get_sans_from_cert(f.read())
new_certr, new_chain, new_key, _ = le_client.obtain_certificate(sans)
if new_chain:
# XXX: Assumes that there was no key change. We need logic
# XXX: Assumes that there was a key change. We need logic
# for figuring out whether there was or not. Probably
# best is to have obtain_certificate return None for
# new_key if the old key is to be used (since save_successor
@@ -180,7 +180,6 @@ def main(config=None, cli_args=sys.argv[1:]):
rc_config = configobj.ConfigObj(cli_config.renewer_config_file)
rc_config.merge(configobj.ConfigObj(
os.path.join(cli_config.renewal_configs_dir, i)))
# TODO: this is a dirty hack!
rc_config.filename = os.path.join(cli_config.renewal_configs_dir, i)
try:
# TODO: Before trying to initialize the RenewableCert object,