Change in vdsm[master]: vdsClient: file based mechanism to provide password

didi at redhat.com didi at redhat.com
Thu Feb 20 07:12:32 UTC 2014


Yedidyah Bar David has posted comments on this change.

Change subject: vdsClient: file based mechanism to provide password
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.ovirt.org/#/c/24733/1//COMMIT_MSG
Commit Message:

Line 9: Current method to provide password for vdsClient commands is typing it
Line 10: in the command line. The BZ1032525 requests to provide safer way.
Line 11: 
Line 12: This patch implements file based way of providing passwords which is
Line 13: encoded using base64 and stored in the first line of the file. Each
Why base64? Why not just plain text? IMO base64 adds no advantages and just complicates use for the common case. We want to make this as comfortable as possible so that people will actually use it if they care even a bit about security.
Line 14: command using passwords now can accept old way password or path to the
Line 15: file with a password.
Line 16: 
Line 17: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I363a16e6a7872ca05e19d5f520bdba90fb492374
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list