Skip to content

Commit

Permalink
Issue 5539 - Make logger's parameter name unified (#5540)
Browse files Browse the repository at this point in the history
Description: Some of the functions of `lib389.cli_conf.security`
used `log` as logger's parameter name while another ones - `logs`.
This lead to regression like #5539.

Fix Description:
Replace `logs` with `log`.

relates: #5539

Reviewed by: mreynolds (Thanks)

(cherry picked from commit b0f8c32)
  • Loading branch information
stanislavlevin authored and progier389 committed Nov 22, 2022
1 parent c3e13e2 commit 12c14ed
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/lib389/lib389/cli_conf/security.py
Expand Up @@ -76,7 +76,7 @@
])


def _security_generic_get(inst, basedn, logs, args, attrs_map):
def _security_generic_get(inst, basedn, log, args, attrs_map):
result = {}
for attr, props in attrs_map.items():
val = props.cls(inst).get_attr_val_utf8(props.attr)
Expand All @@ -89,7 +89,7 @@ def _security_generic_get(inst, basedn, logs, args, attrs_map):
print('\n'.join([f'{attr}: {value or ""}' for attr, value in result.items()]))


def _security_generic_set(inst, basedn, logs, args, attrs_map):
def _security_generic_set(inst, basedn, log, args, attrs_map):
for attr, props in attrs_map.items():
arg = getattr(args, attr.replace('-', '_'))
if arg is None:
Expand Down

0 comments on commit 12c14ed

Please sign in to comment.