Bug 1424892

Summary: Password written in clear text in heat-api.log with DEBUG mode [openstack-rdo]
Product: [Community] RDO Reporter: Summer Long <slong>
Component: openstack-heatAssignee: Amit Ugol <augol>
Status: CLOSED ERRATA QA Contact: Amit Ugol <augol>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: unspecifiedCC: augol, chris.brown, jpeeler, mburns, rhel-osp-director-maint, sbaker, shardy, srevivo, zbitter
Target Milestone: ---Keywords: SecurityTracking
Target Release: trunk   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Release Note
Doc Text:
Story Points: ---
Clone Of: 1424889 Environment:
Last Closed: 2017-08-14 13:51:37 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Summer Long 2017-02-20 02:52:20 UTC 
+++ This bug was initially created as a clone of Bug #1424879 +++

Description of problem:
Reported by Hans Feldt, Ericsson

Password written in clear text in heat-api.log with DEBUG mode
Because this is debug, it is a hardening issue; no CVE is attached.

Upstream bug: https://bugs.launchpad.net/heat/+bug/1664792

Affected code:
heat/common/serializers.py:
 31 class JSONResponseSerializer(object):
 32
 33 def to_json(self, data):
 34 def sanitizer(obj):
 35 if isinstance(obj, datetime.datetime):
 36 return obj.isoformat()
 37 return six.text_type(obj)
 38
 39 response = jsonutils.dumps(data, default=sanitizer)
 40 LOG.debug("JSON response : %s" % response) # <- HERE

Version-Release number of selected component (if applicable):

Steps to Reproduce:
1. Create overcloud
2. View /var/log/heat/heat-api.log
3. Grep for AdminPassword

Actual results:
Plain text is used for passwords

Expected results:
Plain text should never be used for passwords
Comment 1 Zane Bitter 2017-03-16 14:29:51 UTC 
Fix merged upstream.