[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