Francesco Romani has submitted this change and it was merged. Change subject: xmlrpc: Hide fenceNode password in the log ......................................................................
xmlrpc: Hide fenceNode password in the log We enabled logging of fenceNode xmlrpc calls, however there is a plain password in the call parameters. This patch prevents the password from being written to the log. Password logging seems to be already handled with jsonrpc, so this patch just fixes xmlrpc. Change-Id: I53318ed8ce425f042b5f783af779b540c1dfd7b5 Backport-To: 4.0 Backport-To: 3.6 Bug-Url: https://bugzilla.redhat.com/1351247 Signed-off-by: Milan Zamazal <[email protected]> Reviewed-on: https://gerrit.ovirt.org/58833 Reviewed-by: Nir Soffer <[email protected]> Continuous-Integration: Jenkins CI Reviewed-by: Piotr Kliczewski <[email protected]> Reviewed-on: https://gerrit.ovirt.org/59991 Reviewed-by: Francesco Romani <[email protected]> Reviewed-on: https://gerrit.ovirt.org/60096 Reviewed-by: Michal Skrivanek <[email protected]> --- M vdsm/rpc/bindingxmlrpc.py 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Francesco Romani: Looks good to me, approved Michal Skrivanek: Looks good to me, but someone else must approve Milan Zamazal: Verified -- To view, visit https://gerrit.ovirt.org/60096 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I53318ed8ce425f042b5f783af779b540c1dfd7b5 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Milan Zamazal <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: Milan Zamazal <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/admin/lists/[email protected]
