Merge 4cf9964fbdd7130c2e5e35a662b608be37e75ee0 into e597bb208efe942f30139df88b5ef74e78c15925

Co-authored-by: David Welch <david@davidwelch.co>
Co-authored-by: Andre Kradolfer <narfdre@gmail.com>

nvm.sh

@@ -120,15 +120,16 @@ nvm_download() {
     local CURL_COMPRESSED_FLAG=""
     local CURL_HEADER_FLAG=""
 
-    if [ -n "${NVM_AUTH_HEADER:-}" ]; then
-      sanitized_header=$(nvm_sanitize_auth_header "${NVM_AUTH_HEADER}")
-      CURL_HEADER_FLAG="--header \"Authorization: ${sanitized_header}\""
+    if [ -n "$NVM_AUTH_HEADER" ]; then
+        sanitized_header=$(nvm_sanitize_auth_header "$NVM_AUTH_HEADER")
+        CURL_HEADER_FLAG="--header \"Authorization: $sanitized_header\""
     fi
 
     if nvm_curl_use_compression; then
         CURL_COMPRESSED_FLAG="--compressed"
     fi
-    eval "curl -q --fail ${CURL_COMPRESSED_FLAG:-} ${CURL_HEADER_FLAG:-} $*"
+
+    eval "curl -q --fail ${CURL_COMPRESSED_FLAG:-} ${CURL_HEADER_FLAG:-} $@"
   elif nvm_has "wget"; then
     # Emulate curl with wget
     ARGS=$(nvm_echo "$@" | command sed -e 's/--progress-bar /--progress=bar /' \
@@ -151,7 +152,7 @@ nvm_download() {
 
 nvm_sanitize_auth_header() {
     # Remove potentially dangerous characters
-    nvm_echo "$1" | command sed 's/[^a-zA-Z0-9:;_. -]//g'
+    echo "$1" | sed 's/[^a-zA-Z0-9:;_. -]//g'
 }
 
 nvm_has_system_node() {

test/fast/Unit tests/nvm_download

@@ -19,10 +19,9 @@ nvm_download "https://raw.githubusercontent.com/nvm-sh/nvm/HEAD/install.sh" >/de
 # nvm_download should pass when calling with auth header
 docker pull kennethreitz/httpbin && docker run -d --name httpbin -p 80:80 kennethreitz/httpbin
 sleep 1 # wait for httpbin to start
-NVM_AUTH_HEADER="Bearer test-token" nvm_download "http://127.0.0.1/bearer" > /dev/null || die 'nvm_download with auth header should send correctly'
-
-# nvm_download should fail when calling without auth header
-nvm_download "http://127.0.0.1/bearer" > /dev/null && die 'nvm_download with no auth header should not send the header and should fail'
+NVM_AUTH_HEADER="Bearer test-token"
+nvm_download "http://127.0.0.1/bearer" > /dev/null || die "nvm_download unable to send auth header"
+unset NVM_AUTH_HEADER
 docker stop httpbin && docker rm httpbin
 
 cleanup