[pve-devel] [PATCH] remove read_password from CLIHandler

Dominik Csapak d.csapak at proxmox.com
Tue Jun 12 12:33:45 CEST 2018


and replace it with param_mapping, since that implements a very
similar, but more general behaviour

also introduce a get_confirmed_password, as suggested by Thomas[1]

[1]: https://pve.proxmox.com/pipermail/pve-devel/2018-June/032338.html

pve-common:

Dominik Csapak (2):
  remove read_password_func from cli handler
  add a generalized 'read and confirm password' sub

 src/PVE/CLIHandler.pm  | 18 ++++++++----------
 src/PVE/JSONSchema.pm  | 15 +--------------
 src/PVE/PTY.pm         |  7 +++++++
 src/PVE/RESTHandler.pm | 26 +++++++++++++-------------
 4 files changed, 29 insertions(+), 37 deletions(-)

pve-access-control:

Dominik Csapak (2):
  fix typo in change_passsword
  replace read_password with param_mapping

 PVE/API2/AccessControl.pm |  2 +-
 PVE/CLI/pveum.pm          | 32 +++++++++++++++++++++-----------
 debian/control            |  1 -
 test/auth-test.pl         | 12 ++----------
 4 files changed, 24 insertions(+), 23 deletions(-)

pve-storage:

Dominik Csapak (1):
  replace read_password with param_mapping

 PVE/CLI/pvesm.pm | 15 +++++++++++++--
 1 file changed, 13 insertions(+), 2 deletions(-)

qemu-server:

Dominik Csapak (1):
  use get_confirmed_password

 PVE/CLI/qm.pm | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

pve-container:

Dominik Csapak (1):
  replace read_password with param_mapping

 src/PVE/CLI/pct.pm | 23 ++++++++++++++---------
 1 file changed, 14 insertions(+), 9 deletions(-)

-- 
2.11.0





More information about the pve-devel mailing list