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

Dominik Csapak d.csapak at proxmox.com
Fri Jun 15 15:28:40 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

this further implements a 'get_standard_mapping' sub to
reuse parameter mappings we often use (like for the password)

pve-common:

Dominik Csapak (4):
  remove read_password_func from cli handler
  add the possibility to use a hash for parameter mapping
  CLIHandler: add standard_mappings
  CLIHandler: add standard mapping for password parameter

 src/PVE/CLIHandler.pm  | 48 ++++++++++++++++++++++++++++++++++++++----------
 src/PVE/JSONSchema.pm  | 15 +--------------
 src/PVE/RESTHandler.pm | 36 +++++++++++++++++++-----------------
 3 files changed, 58 insertions(+), 41 deletions(-)

pve-manager:

Dominik Csapak (2):
  replace read_password with param_mapping in pvesh
  use PVE::RESTHandler explicitly

 bin/pvesh | 25 +++++++------------------
 1 file changed, 7 insertions(+), 18 deletions(-)

pve-access-control:

Dominik Csapak (1):
  replace read_password with param_mapping

 PVE/CLI/pveum.pm  | 28 ++++++++++++++++++----------
 debian/control    |  1 -
 test/auth-test.pl | 12 ++----------
 3 files changed, 20 insertions(+), 21 deletions(-)

pve-storage:

Dominik Csapak (1):
  replace read_password with param_mapping

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

pve-container:

Dominik Csapak (1):
  replace read_password with param_mapping

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

qemu-server:

Dominik Csapak (1):
  use get_standard_mapping for cipassword_map

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

-- 
2.11.0




More information about the pve-devel mailing list