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

@@ -35,40 +35,42 @@ else
     fi
 fi
 
+# Redis connection check function
 check_redis() {
-    redis-cli -h "${REDIS_HOST}" -p "${REDIS_PORT}" -a "${REDIS_PASSWORD}" ping > /dev/null 2>&1
+    local max_attempts=30
+    local attempt=1
+    
+    echo "Waiting for Redis to be ready..."
+    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
+            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))
+        sleep 5
+        
+        if [ $attempt -eq $max_attempts ]; then
+            echo "Falling back to local Redis..."
+            export REDIS_TYPE="local"
+        fi
+    done
+    return 1
 }
 
-check_redis() {
-    if [ "${REDIS_TYPE}" = "local" ]; then
-        echo "Starting local Redis server..."
-        redis-server --daemonize yes --requirepass "${REDIS_PASSWORD}"
-        sleep 2
-        return 0
-    else
-        redis-cli -h "${REDIS_HOST}" -p "${REDIS_PORT}" -a "${REDIS_PASSWORD}" ping > /dev/null 2>&1
-    fi
+# Try to connect to Redis
+check_redis || {
+    echo "Failed to connect to Redis after all attempts"
+    exit 1
 }
 
-echo "Waiting for Redis to be ready..."
-local max_attempts=30
-local attempt=1
-
-while [ $attempt -le $max_attempts ]; do
-    if check_redis; then
-        echo "Redis is ready!"
-        break
-    fi
-    echo "Redis is unavailable (attempt $attempt/$max_attempts) - retrying..."
-    attempt=$((attempt + 1))
-    sleep 5
-    
-    if [ $attempt -eq $max_attempts ]; then
-        echo "Falling back to local Redis..."
-        export REDIS_TYPE="local"
-    fi
-done
-
 # Initialize database if needed
 cd /app/api
 if [ ! -f ".db_initialized" ]; then