Improve __sdkman_secure_curl_download (#763)

* Use shell-agnostic code in …_secure_curl_download
* Replicate original order of curl arguments

Author: rgoldberg

src/main/bash/sdkman-utils.sh

@@ -31,32 +31,30 @@ function __sdkman_secure_curl() {
 }
 
 function __sdkman_secure_curl_download() {
-	local curl_params="--progress-bar --location"
-	if [[ "${sdkman_insecure_ssl}" == 'true' ]]; then
-		curl_params="$curl_params --insecure"
-	fi
+	local curl_params
+	curl_params=('--progress-bar' '--location')
 
-	if [[ ! -z "${sdkman_curl_retry}" ]]; then
-		curl_params="--retry ${sdkman_curl_retry} ${curl_params}"
+	if [[ "${sdkman_debug_mode}" == 'true' ]]; then
+		curl_params+=('--verbose')
 	fi
 
-	if [[ ! -z "${sdkman_curl_retry_max_time}" ]]; then
-		curl_params="--retry-max-time ${sdkman_curl_retry_max_time} ${curl_params}"
+	if [[ "${sdkman_curl_continue}" == 'true' ]]; then
+		curl_params+=('-C' '-')
 	fi
 
-	if [[ "${sdkman_curl_continue}" == 'true' ]]; then
-		curl_params="-C - ${curl_params}"
+	if [[ -n "${sdkman_curl_retry_max_time}" ]]; then
+		curl_params+=('--retry-max-time' "${sdkman_curl_retry_max_time}")
 	fi
 
-	if [[ "${sdkman_debug_mode}" == 'true' ]]; then
-		curl_params="--verbose ${curl_params}"
+	if [[ -n "${sdkman_curl_retry}" ]]; then
+		curl_params+=('--retry' "${sdkman_curl_retry}")
 	fi
 
-	if [[ "$zsh_shell" == 'true' ]]; then
-		curl ${=curl_params} "$@"
-	else
-		curl ${curl_params} "$@"
+	if [[ "${sdkman_insecure_ssl}" == 'true' ]]; then
+		curl_params+=('--insecure')
 	fi
+
+	curl "${curl_params[@]}" "${@}"
 }
 
 function __sdkman_secure_curl_with_timeouts() {