Skip to content

Commit

Permalink
pki: T6241: remove debug print statement about updated subsystems (#3670
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.

(cherry picked from commit a4d49a9)

Co-authored-by: Christian Breunig <christian@breunig.cc>
  • Loading branch information
mergify[bot] and c-po committed Jun 24, 2024
1 parent a109228 commit 6508db4
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 6508db4

Please sign in to comment.