WIP: hostname-vars: Ensure previous hostname envvars aren't overwritten

Connects-to: #78 Change-type: patch Signed-off-by: Heds Simons <[email protected]>
balena-io-modules · Mar 14, 2019 · decdaab · decdaab
1 parent a4b121f
commit decdaab
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/configure-balena-host-envvars.sh b/src/configure-balena-host-envvars.sh
@@ -36,17 +36,17 @@ HOST_ENVVARS[BALENA_SENTRY_URL_HOST]=sentry
 HOST_ENVVARS[BALENA_MONITOR_HOST]=monitor
 
 # Go through the lists and fill in any missing envvars
-#for index in $(seq 1 ${#HOST_ENVVARS[*]}); do
 for VARNAME in "${!HOST_ENVVARS[@]}"; do
-    VARVALUE="${HOST_ENVVARS[$VARNAME]}"
-    if [[ ! -z "$VARVALUE" ]]; then
+    VARVALUE=${!VARNAME}
+    if [[ -z "$VARVALUE" ]]; then
+        PREFIX="${HOST_ENVVARS[$VARNAME]}"
         # Only use BALENA_DEVICE_UUID if it actually exists, else just use the
         # full passed in TLD
         DEVICE=""
         if [[ ! -z "$BALENA_DEVICE_UUID" ]]; then
             DEVICE="$BALENA_DEVICE_UUID."
         fi
-        SUBDOMAIN="$VARVALUE.$DEVICE$BALENA_TLD"
+        SUBDOMAIN="$PREFIX.$DEVICE$BALENA_TLD"
 
         # Several vars require special formatting
         if [ "$VARNAME" == "BALENA_TOKEN_AUTH_REALM" ]; then