openstack/nova
Code Issues Proposed changes

Remove unnecessary credential sanitation for logging

Browse Source 
Now that the credentials dict is the return from
context.to_policy_values() and not the context object itself there's no
need to attempt to pop the 'auth_token' key out before logging.

Change-Id: I3b4eacf6c7d97a4243aff026aefbb53017d5e840
This commit is contained in:
Andrew Laski 2016-10-05 12:54:23 -04:00
parent b796673f39
commit b713a62740
1 changed files with 0 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
nova/policy.py
View File

@ -162,7 +162,6 @@ def authorize(context, action, target, do_raise=True, exc=None):
with excutils.save_and_reraise_exception():
LOG.exception(_LE('Policy not registered'))
except Exception:
credentials.pop('auth_token', None)
with excutils.save_and_reraise_exception():
LOG.debug('Policy check for %(action)s failed with credentials '
'%(credentials)s',