openstack/python-openstackclient
Code Issues Proposed changes

Mask passwords in debug logs for auth_config_hook

Browse Source 
The auth config hook can have credentials in it so
we have to mask the config before logging it. To
avoid doing the work of masking the password if we
aren't going to log it, there is a conditional put
around the actual debug statement.

Change-Id: I8e626672ec94fc837610216bccb4354dbdedca17
Closes-Bug: #1630822
This commit is contained in:
Matt Riedemann 2016-10-05 21:11:16 -04:00
parent bfeecd50fd
commit cd1a412408
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
openstackclient/common/client_config.py
View File

@ -17,6 +17,8 @@ import logging
from os_client_config import config
from os_client_config import exceptions as occ_exceptions
from oslo_utils import strutils
import six
LOG = logging.getLogger(__name__)
@ -180,7 +182,9 @@ class OSC_Config(config.OpenStackConfig):
config = self._auth_v2_ignore_v3(config)
config = self._auth_default_domain(config)
LOG.debug("auth_config_hook(): %s" % config)
if LOG.isEnabledFor(logging.DEBUG):
LOG.debug("auth_config_hook(): %s",
strutils.mask_password(six.text_type(config)))
return config
def load_auth_plugin(self, config):