From 800fd91353a18fd17d4113e38a4835691fd1962e Mon Sep 17 00:00:00 2001 From: Orazio Date: Mon, 6 Jan 2020 11:53:29 +0100 Subject: [PATCH] Use radiolist to select a VPN --- auto_install/install.sh | 18 +++++++++++------- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/auto_install/install.sh b/auto_install/install.sh index c37a30c..7a77fbc 100755 --- a/auto_install/install.sh +++ b/auto_install/install.sh @@ -225,7 +225,7 @@ askAboutExistingInstall(){ "${opt3a}" "${opt3b}" 3>&2 2>&1 1>&3) || \ { echo "::: Cancel selected. Exiting"; exit 1; } - echo "::: ${opt1a} option selected." + echo "::: ${UpdateCmd} option selected." } @@ -252,8 +252,7 @@ distroCheck(){ Debian|Raspbian|Ubuntu) case ${OSCN} in buster|bionic) - # shellcheck disable=SC2104 - break + : ;; *) maybeOSSupport @@ -883,11 +882,16 @@ askWhichVPN(){ fi fi else - if (whiptail --backtitle "Setup PiVPN" --title "Installation mode" --yesno "WireGuard is a new kind of VPN that provides near-istantaneous connection speed, high performance, modern cryptography.\\n\\nIt's the recommended choice expecially if you use mobile devices where WireGuard is easier on battery than OpenVPN.\\n\\nOpenVPN is still available if you need the traditional, flexible, trusted VPN protocol. Or if you need features like TCP and custom search domain.\\n\\nChoose 'Yes' to use WireGuard or 'No' to use OpenVPN." ${r} ${c}); - then - VPN="wireguard" + chooseVPNCmd=(whiptail --backtitle "Setup PiVPN" --title "Installation mode" --separate-output --radiolist "WireGuard is a new kind of VPN that provides near-istantaneous connection speed, high performance, modern cryptography.\\n\\nIt's the recommended choice expecially if you use mobile devices where WireGuard is easier on battery than OpenVPN.\\n\\nOpenVPN is still available if you need the traditional, flexible, trusted VPN protocol. Or if you need features like TCP and custom search domain.\\n\\nChoose a VPN (press space to select):" "${r}" "${c}" 2) + VPNChooseOptions=(WireGuard "" on + OpenVPN "" off) + + if VPN=$("${chooseVPNCmd[@]}" "${VPNChooseOptions[@]}" 2>&1 >/dev/tty) ; then + echo "::: Using VPN: $VPN" + VPN="${VPN,,}" else - VPN="openvpn" + echo "::: Cancel selected, exiting...." + exit 1 fi fi