Merge pull request #601 from FalkorDB/fix/skip-internal-falkordb-when-external

fix: skip internal FalkorDB when FALKORDB_HOST is external

Author: gkorland

start.sh

@@ -5,13 +5,21 @@ set -e
 FALKORDB_HOST="${FALKORDB_HOST:-localhost}"
 FALKORDB_PORT="${FALKORDB_PORT:-6379}"
 
-# Start FalkorDB Redis server in background
-redis-server --loadmodule /var/lib/falkordb/bin/falkordb.so | cat &
+# Start FalkorDB Redis server in background only if using a local address (not an external instance)
+if [ "${FALKORDB_HOST}" = "localhost" ] || [[ "${FALKORDB_HOST}" =~ ^127\.0\.0\.[0-9]+$ ]]; then
+    redis-server --loadmodule /var/lib/falkordb/bin/falkordb.so | cat &
+fi
 
 # Wait until FalkorDB is ready
-echo "Waiting for FalkorDB to start on $FALKORDB_HOST:$FALKORDB_PORT..."
+FALKORDB_WAIT_TIMEOUT="${FALKORDB_WAIT_TIMEOUT:-30}"
+echo "Waiting for FalkorDB to start on $FALKORDB_HOST:$FALKORDB_PORT (timeout: ${FALKORDB_WAIT_TIMEOUT}s)..."
 
+SECONDS=0
 while ! nc -z "$FALKORDB_HOST" "$FALKORDB_PORT"; do
+  if [ "$SECONDS" -ge "$FALKORDB_WAIT_TIMEOUT" ]; then
+    echo "ERROR: FalkorDB did not become reachable at $FALKORDB_HOST:$FALKORDB_PORT within ${FALKORDB_WAIT_TIMEOUT}s" >&2
+    exit 1
+  fi
   sleep 0.5
 done