Murano

Make rabbitmq password config property as secret

Bug #1658044 reported by zhurong
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Murano
Fix Released
Low
zhurong

Bug Description

Set `secret=True` so that its value will not be printed in logs to avoid
some security problems [1].

[1] https://github.com/openstack/oslo.config/blob/master/oslo_config/cfg.py#L767

Revision history for this message
zhurong (zhu-rong) wrote :

Fix proposed to branch: master
Review: https://review.openstack.org/#/c/421755/

Changed in murano:
importance: Undecided → Low
status: New → Confirmed
assignee: nobody → zhurong (zhu-rong)
OpenStack Infra (hudson-openstack)
Changed in murano:
status: Confirmed → In Progress
Revision history for this message
zhurong (zhu-rong) wrote :

Fix proposed to branch: master
https://review.openstack.org/#/c/422327/

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to murano-agent (master)

Reviewed: https://review.openstack.org/422327
Committed: https://git.openstack.org/cgit/openstack/murano-agent/commit/?id=8739083d09ad459a5b2aa156705a9b8569303864
Submitter: Jenkins
Branch: master

commit 8739083d09ad459a5b2aa156705a9b8569303864
Author: zhurong <email address hidden>
Date: Thu Jan 19 09:11:00 2017 +0800

    Marking rabbitmq password config property as secret

    As this has changed in murano in [0],
    this patch change murano-agent.

    [0]: I9f41f334e0f32ee31c40c27c977ed801a68830e7

    Closes-bug: #1658044
    Change-Id: Idc80808dcf54eb2d457cdcf3d42e23236e07a64f

Changed in murano:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/murano-agent 3.1.0

This issue was fixed in the openstack/murano-agent 3.1.0 release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.