fix: Remove redundant exports from deploy workflow

Author: milanmajchrak

.github/workflows/deploy.yml

@@ -58,9 +58,6 @@ jobs:
       - name: deploy to dev-5
         working-directory: build-scripts/run/
         run: |
-          export INSTANCE="${INSTANCE}"
-          export ADMIN_PASSWORD="${ADMIN_PASSWORD}"
-          export USER_PASSWORD="${USER_PASSWORD}"
           ./start.sh dspace-$INSTANCE
           cd ../..
           # use customized docker-compose file for the `85` port because the server is exposed with the namespace `repository`
@@ -92,9 +89,6 @@ jobs:
       - name: deploy dspace-import on dev-5
         working-directory: build-scripts/run/
         run: |
-          export INSTANCE="${INSTANCE}"
-          export ADMIN_PASSWORD="${ADMIN_PASSWORD}"
-          export USER_PASSWORD="${USER_PASSWORD}"
           ./start.sh dspace-$INSTANCE
           cd ../..
           # this is not necessary, since extra.yml doesn't contain any new images that weren't pulled within script above

build-scripts/run/start.sh

@@ -8,6 +8,11 @@ PROJECT=${1:-unnamed_dspace}
 
 echo "Using envfile: [$ENVFILE] for project: [$PROJECT]"
 
+# Debug: Check what's in the environment before anything
+echo "DEBUG: Environment check before sourcing:"
+echo "  ADMIN_PASSWORD in env: $(printenv ADMIN_PASSWORD > /dev/null 2>&1 && echo 'yes' || echo 'NO')"
+echo "  USER_PASSWORD in env: $(printenv USER_PASSWORD > /dev/null 2>&1 && echo 'yes' || echo 'NO')"
+
 # Preserve passwords if they're already set - they take precedence over env file
 SAVED_ADMIN_PASSWORD="${ADMIN_PASSWORD:-}"
 SAVED_USER_PASSWORD="${USER_PASSWORD:-}"