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

iheim at redhat.com iheim at redhat.com
Mon Oct 7 20:22:39 UTC 2013


Itamar Heim 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:
  Itamar Heim: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I81c7371eaf930b3cc21861deb620c61891ad4d1f
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.3
Gerrit-Owner: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: Itamar Heim <iheim 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>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list