summaryrefslogtreecommitdiff
path: root/src/conf_mode/pki.py
diff options
context:
space:
mode:
Diffstat (limited to 'src/conf_mode/pki.py')
-rwxr-xr-xsrc/conf_mode/pki.py15
1 files changed, 8 insertions, 7 deletions
diff --git a/src/conf_mode/pki.py b/src/conf_mode/pki.py
index c1ff80d8a..98922595c 100755
--- a/src/conf_mode/pki.py
+++ b/src/conf_mode/pki.py
@@ -231,7 +231,7 @@ def get_config(config=None):
path = search['path']
path_str = ' '.join(path + found_path).replace('_','-')
- #print(f'PKI: Updating config: {path_str} {item_name}')
+ print(f'PKI: Updating config: {path_str} {item_name}')
if path[0] == 'interfaces':
ifname = found_path[0]
@@ -241,10 +241,10 @@ def get_config(config=None):
if not D.node_changed_presence(path):
set_dependents(path[1], conf)
- # Check PKI certificates if they are generated by ACME. If they are, traverse
- # the current configutration and determine the service where the certificate
- # is used by. This is needed to check if we might need to start ACME behing
- # a reverse proxy.
+ # Check PKI certificates if they are auto-generated by ACME. If they are,
+ # traverse the current configuration and determine the service where the
+ # certificate is used by.
+ # Required to check if we might need to run certbot behing a reverse proxy.
if 'certificate' in pki:
for name, cert_config in pki['certificate'].items():
if 'acme' not in cert_config:
@@ -252,7 +252,7 @@ def get_config(config=None):
if not dict_search('system.load_balancing.haproxy', pki):
continue
used_by = []
- for cert_list, cli_path in dict_search_recursive(
+ for cert_list, _ in dict_search_recursive(
pki['system']['load_balancing']['haproxy'], 'certificate'):
if name in cert_list:
used_by.append('haproxy')
@@ -356,7 +356,8 @@ def verify(pki):
if 'used_by' not in cert_conf['acme']:
if not check_port_availability(listen_address, 80):
- raise ConfigError(f'Port 80 is not available for ACME challenge for certificate "{name}"!')
+ raise ConfigError('Port 80 is already in use and not available '\
+ f'to provide ACME challenge for "{name}"!')
if 'certbot_renew' not in pki:
# Only run the ACME command if something on this entity changed,