Skip to content

Commit

Permalink
pki: T6241: remove debug print statement about updated subsystems
Browse files Browse the repository at this point in the history
Commit 9f9891a ("pki: T6241: Fix dependency updates on PKI changes") added
a print() statement which notified the users about the subsystems which got
supplied with an updated certificate.

Example:
> PKI: Updating config: interfaces openvpn vtun0 tls certificate openvpn_vtun0
> PKI: Updating config: interfaces openvpn vtun0 tls ca_certificate openvpn_vtun0_1

This is an informational message which should maybe (if needed) be sent to
syslog. But the main issue is that CLI paths are mangled (- to _) which makes
the about print output wrong and could potentially confuse users.

Statement has been commented to be re-enabled for debugging.
  • Loading branch information
c-po committed Jun 15, 2024
1 parent f3d3b0f commit a4d49a9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/conf_mode/pki.py
Original file line number Diff line number Diff line change
Expand Up @@ -232,7 +232,7 @@ def get_config(config=None):

path = search['path']
path_str = ' '.join(path + found_path)
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]
Expand Down

0 comments on commit a4d49a9

Please sign in to comment.