From: Dietmar Maurer <dietmar@proxmox.com>
Date: Wed, 6 Jun 2018 05:02:36 +0000 (+0200)
Subject: allow to setup remotes without password
X-Git-Url: https://git.proxmox.com/?p=pve-client.git;a=commitdiff_plain;h=8842464b81a452996373e5d21205282eeef66888

allow to setup remotes without password

And query the password on demand.
---

diff --git a/PVE/APIClient/Commands/remote.pm b/PVE/APIClient/Commands/remote.pm
index e8c876a..781fb63 100644
--- a/PVE/APIClient/Commands/remote.pm
+++ b/PVE/APIClient/Commands/remote.pm
@@ -61,9 +61,14 @@ __PACKAGE__->register_method ({
 
 	my $last_fp = 0;
 
+	my $password = $param->{password};
+	if (!defined($password)) {
+	    $password = PVE::PTY::read_password("Remote password: ");
+	}
+
 	my $setup = {
 	    username                => $param->{username},
-	    password                => $param->{password},
+	    password                => $password,
 	    host                    => $param->{host},
 	    port                    => $param->{port} // 8006,
 	};
diff --git a/PVE/APIClient/Config.pm b/PVE/APIClient/Config.pm
index 65c555f..8fa7691 100644
--- a/PVE/APIClient/Config.pm
+++ b/PVE/APIClient/Config.pm
@@ -76,7 +76,7 @@ sub options {
 	host => { optional => 0 },
 	comment => { optional => 1 },
 	username => { optional => 0 },
-	password => { optional => 0 },
+	password => { optional => 1 },
 	port => { optional => 1 },
 	fingerprint => { optional => 1 },
    };
@@ -138,11 +138,17 @@ sub remote_conn {
     my ($class, $cfg, $remote) = @_;
 
     my $section = $class->lookup_remote($cfg, $remote);
+
+    my $password = $section->{password};
+    if (!defined($password)) {
+	$password = PVE::PTY::read_password("Remote password: ")
+    }
+
     my $conn = PVE::APIClient::LWP->new(
 	username                => $section->{username},
-	password                => $section->{password},
+	password                => $password,
 	host                    => $section->{host},
-	port                    => $section->{port},
+	port                    => $section->{port} // 8006,
 	cached_fingerprints     => {
 	    $section->{fingerprint} => 1,
 	}