From a846278b6dc2c6074713f43cc7cc0e2ea7ab3b6b Mon Sep 17 00:00:00 2001 From: Arthur Dayne Date: Fri, 3 Apr 2020 14:42:39 +0800 Subject: [PATCH] Remove the conversion according to the comment of jdennis Because the bug #1804528 has been fixed, the conversion to dict can be removed. Change-Id: Ibec9ec21096977c2876b373e388647766c79b3a7 --- oslo_policy/policy.py | 15 +-------------- 1 file changed, 1 insertion(+), 14 deletions(-) diff --git a/oslo_policy/policy.py b/oslo_policy/policy.py index 61a3597d..c1dca53b 100644 --- a/oslo_policy/policy.py +++ b/oslo_policy/policy.py @@ -878,20 +878,7 @@ class Enforcer(object): if LOG.isEnabledFor(logging.DEBUG): try: - # NOTE(jdennis) Although a MutableMapping behaves like - # a dict oslo.strutils.mask_dict_password() requires a - # dict. Bug #1804528 was opened to fix this, once that - # bug is fixed the conversion to dict can be removed. - if isinstance(creds, dict): - creds_dict = creds - elif isinstance(creds, collections.MutableMapping): - creds_dict = dict(creds) - else: - raise TypeError('unexpected type %(creds_type)s' % - {'creds_type': type(creds)}) - creds_dict = strutils.mask_dict_password( - copy.deepcopy(creds_dict) - ) + creds_dict = strutils.mask_dict_password(copy.deepcopy(creds)) creds_msg = jsonutils.dumps(creds_dict, skipkeys=True, sort_keys=True) except Exception as e: