fix: improve Redis and PostgreSQL connection handling

- Add proper error messages for connection failures
- Implement local Redis fallback mechanism
- Fix variable scoping in Redis check function
- Add detailed logging for connection attempts
- Ensure proper exit codes on failures

docker/entrypoint.sh

@@ -6,11 +6,9 @@ echo "Starting Dify services..."
 # Database connection check based on type
 if [ "${DATABASE_TYPE}" = "sqlite" ]; then
     echo "Using SQLite database at ${SQLITE_PATH}"
-    # Ensure directory exists
     mkdir -p $(dirname "${SQLITE_PATH}")
     touch "${SQLITE_PATH}"
 else
-    # Enhanced PostgreSQL connection check with timeout
     check_postgres() {
         local max_attempts=30
         local attempt=1
@@ -26,7 +24,6 @@ else
         return 1
     }
 
-    # Try to connect with timeout
     if ! check_postgres; then
         echo "Failed to connect to PostgreSQL, falling back to SQLite..."
         export DATABASE_TYPE="sqlite"
@@ -35,23 +32,27 @@ else
     fi
 fi
 
-# Redis connection check function
+# Redis connection check function with HF Spaces compatibility
 check_redis() {
     local max_attempts=30
     local attempt=1
     
-    echo "Waiting for Redis to be ready..."
+    # For HF Spaces, start local Redis by default
+    if [ "${REDIS_TYPE}" = "local" ] || [ "${SPACE_ID}" != "" ]; then
+        echo "Starting local Redis server..."
+        redis-server --daemonize yes --port 6379 --requirepass "${REDIS_PASSWORD:-difyai123456}"
+        export REDIS_HOST=localhost
+        sleep 2
+        if redis-cli -h localhost -p 6379 -a "${REDIS_PASSWORD:-difyai123456}" ping > /dev/null 2>&1; then
+            echo "Local Redis server started successfully"
+            return 0
+        fi
+    fi
+    
     while [ $attempt -le $max_attempts ]; do
-        if [ "${REDIS_TYPE}" = "local" ]; then
-            echo "Starting local Redis server..."
-            redis-server --daemonize yes --requirepass "${REDIS_PASSWORD}"
-            sleep 2
+        if redis-cli -h "${REDIS_HOST}" -p "${REDIS_PORT}" -a "${REDIS_PASSWORD}" ping > /dev/null 2>&1; then
+            echo "Redis is ready!"
             return 0
-        else
-            if redis-cli -h "${REDIS_HOST}" -p "${REDIS_PORT}" -a "${REDIS_PASSWORD}" ping > /dev/null 2>&1; then
-                echo "Redis is ready!"
-                return 0
-            fi
         fi
         echo "Redis is unavailable (attempt $attempt/$max_attempts) - retrying..."
         attempt=$((attempt + 1))
@@ -60,6 +61,13 @@ check_redis() {
         if [ $attempt -eq $max_attempts ]; then
             echo "Falling back to local Redis..."
             export REDIS_TYPE="local"
+            export REDIS_HOST=localhost
+            redis-server --daemonize yes --port 6379 --requirepass "${REDIS_PASSWORD:-difyai123456}"
+            sleep 2
+            if redis-cli -h localhost -p 6379 -a "${REDIS_PASSWORD:-difyai123456}" ping > /dev/null 2>&1; then
+                echo "Local Redis server started successfully"
+                return 0
+            fi
         fi
     done
     return 1