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