Skip to content

Commit

Permalink
clean-up for KC-788 (security-audit-report): remove unused variable
Browse files Browse the repository at this point in the history
  • Loading branch information
aaunario-keeper committed Jul 3, 2024
1 parent 43c5b0f commit b32cf5a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions keepercommander/commands/security_audit.py
Original file line number Diff line number Diff line change
Expand Up @@ -174,7 +174,7 @@ def get_node_id(name_or_id):
node = next(iter(matches)) if matches else {}
return node.get('node_id')

def report_errors(emails):
def report_errors():
title = 'Security Audit Report - Problems Found\nSecurity data could not be parsed for the following vaults:'
output_fmt = kwargs.get('format', 'table')
headers = ['vault_owner', 'error_message']
Expand Down Expand Up @@ -303,7 +303,7 @@ def update_vault_errors(username, error):
rows.append(row)

if vault_errors_lookup.keys():
return report_errors(vault_errors_lookup)
return report_errors()

if save_report:
self.save_updated_security_reports(params, updated_security_reports)
Expand Down

0 comments on commit b32cf5a

Please sign in to comment.