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 _______________________________________________ pve-devel mailing list pve-devel@pve.proxmox.com https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel