Skip to content

Commit 4f3f1eb

Browse files
[auto-install] Fix case sensitivity for OpenVPN disable option #490
Replace bash 4.0+ specific ${var,,} syntax with explicit case checking to ensure compatibility with older bash versions (3.x). Now accepts both 'N' and 'n' to disable OpenVPN module as advertised in the prompt. Fixes issue where users entering uppercase 'N' (as instructed) would not disable the OpenVPN module on systems with bash < 4.0.
1 parent 244e196 commit 4f3f1eb

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

deploy/auto-install.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -148,7 +148,7 @@ setup_docker_openwisp() {
148148
# VPN domain
149149
if [[ -z "$vpn_domain" ]]; then
150150
set_env "VPN_DOMAIN" "openvpn.${domain}"
151-
elif [[ "${vpn_domain,,}" == "n" ]]; then
151+
elif [[ "$vpn_domain" == "n" || "$vpn_domain" == "N" ]]; then
152152
set_env "VPN_DOMAIN" ""
153153
else
154154
set_env "VPN_DOMAIN" "$vpn_domain"

0 commit comments

Comments
 (0)