Change in ovirt-engine[master]: packaging: setup: Do not log passwords

sbonazzo at redhat.com sbonazzo at redhat.com
Mon Oct 7 14:35:14 UTC 2013


Sandro Bonazzola has submitted this change and it was merged.

Change subject: packaging: setup: Do not log passwords
......................................................................


packaging: setup: Do not log passwords

Most of the code already refrains from logging passwords.
This change also makes upgrade from legacy 3.2 code not log the db
password and also makes otopi not log db password and admin password
in dumpEnvironment even if set from an answer file etc.
Depends on http://gerrit.ovirt.org/19776 .

Bug-Url: https://bugzilla.redhat.com/1014115
Change-Id: I81c7371eaf930b3cc21861deb620c61891ad4d1f
Signed-off-by: Yedidyah Bar David <didi at redhat.com>
---
M ovirt-engine.spec.in
M packaging/setup/plugins/ovirt-engine-common/db/connection.py
M packaging/setup/plugins/ovirt-engine-setup/config/options.py
M packaging/setup/plugins/ovirt-engine-setup/legacy/database.py
4 files changed, 27 insertions(+), 1 deletion(-)

Approvals:
  Sandro Bonazzola: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/19775
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I81c7371eaf930b3cc21861deb620c61891ad4d1f
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschreib at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>



More information about the Engine-commits mailing list