@pioneer-platform/default-redis 8.11.0 → 8.11.2

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @pioneer-platform/default-redis
 
+## 8.11.2
+
+### Patch Changes
+
+- fix: enable Redis offline queue to prevent startup crashes
+
+## 8.11.1
+
+### Patch Changes
+
+- cache work
+
 ## 8.11.0
 
 ### Minor Changes

package/index.js

@@ -7,29 +7,82 @@
 const TAG = " | REDIS-CONNECTION-MODULE | "
 //const log = require('@pioneer-platform/loggerdog')()
 
-//mock in tests
-let Redis
-if(process.env.NODE_ENV === 'test'){
-	Redis = require('ioredis-mock');
-}else{
-	Redis = require("ioredis");
-}
-
+const Redis = require("ioredis");
 
 let redis
-let redisQueue
+let redisQueue  // Dedicated client for blocking operations (brpop, blpop, etc.)
 let publisher
 let subscriber
 
+// Redis connection config with IPv4 enforcement and extended timeouts
+// IPv4 enforcement prevents 10-second IPv6 timeout fallback
+// Extended timeouts accommodate DNS resolution and network latency
+const redisConfig = {
+    host: '127.0.0.1',        // Force IPv4, not "localhost" which tries ::1 first
+    port: 6379,
+    family: 4,                 // Force IPv4 socket family
+    connectTimeout: 10000,     // Increased from 1500ms to 10000ms for reliable connection
+    commandTimeout: 10000,     // Timeout for individual Redis commands (GET, SET, etc.)
+    keepAlive: 5000,           // TCP keep-alive interval
+    maxRetriesPerRequest: 3,   // Increased from 1 to 3 - allow retries for transient network issues
+    enableReadyCheck: true,
+    enableOfflineQueue: true,  // Queue commands when disconnected to prevent crashes during startup
+    retryStrategy(times) {
+        const delay = Math.min(times * 50, 2000);
+        console.log(TAG, `Redis reconnect attempt ${times} in ${delay}ms`);
+        return delay;
+    },
+};
+
+// Override with connection string if provided
+if (process.env.REDIS_CONNECTION) {
+    console.log(TAG, "Using REDIS_CONNECTION from env:", process.env.REDIS_CONNECTION);
+    const connUrl = new URL(process.env.REDIS_CONNECTION);
+
+    // Merge with base config to preserve timeouts and IPv4 enforcement
+    Object.assign(redisConfig, {
+        host: connUrl.hostname,
+        port: parseInt(connUrl.port) || 6379,
+        password: connUrl.password || undefined,
+        username: (connUrl.username && connUrl.username !== 'default') ? connUrl.username : undefined,
+    });
 
-if(!process.env.REDIS_CONNECTION) {
-	console.log("Looking for redis on: redis://127.0.0.1:6379")
-	process.env.REDIS_CONNECTION = "redis://127.0.0.1:6379"
+    console.log(TAG, "Merged config with IPv4 enforcement:", {
+        host: redisConfig.host,
+        port: redisConfig.port,
+        family: redisConfig.family,
+        connectTimeout: redisConfig.connectTimeout,
+        commandTimeout: redisConfig.commandTimeout
+    });
+} else {
+    console.log(TAG, "Connecting to Redis on 127.0.0.1:6379 (IPv4 only)");
 }
 
-redis = new Redis(process.env.REDIS_CONNECTION)
-publisher = new Redis(process.env.REDIS_CONNECTION)
-subscriber = new Redis(process.env.REDIS_CONNECTION)
-redisQueue = new Redis(process.env.REDIS_CONNECTION)
+// Create Redis clients
+redis = new Redis(redisConfig);
+publisher = new Redis(redisConfig);
+subscriber = new Redis(redisConfig);
+
+// CRITICAL: Dedicated client for blocking operations (brpop, blpop, bzpopmin, etc.)
+// Blocking operations on the main redis client will block ALL cache read/write operations
+// This was the root cause of 2-10 second cache GET timeouts
+redisQueue = new Redis(redisConfig);
+
+// Event handlers
+redis.on('connect', () => console.log(TAG, 'Main Redis client connected'));
+redis.on('ready', () => console.log(TAG, 'Main Redis client ready'));
+redis.on('error', (err) => console.error(TAG, 'Main Redis error:', err.message));
+
+redisQueue.on('connect', () => console.log(TAG, 'Queue Redis client connected'));
+redisQueue.on('ready', () => console.log(TAG, 'Queue Redis client ready'));
+redisQueue.on('error', (err) => console.error(TAG, 'Queue Redis error:', err.message));
+
+publisher.on('connect', () => console.log(TAG, 'Publisher client connected'));
+subscriber.on('connect', () => console.log(TAG, 'Subscriber client connected'));
 
-module.exports = {redis, publisher, subscriber, redisQueue}
+module.exports = {
+	redis,
+	publisher,
+	subscriber,
+	redisQueue  // Export dedicated queue client for blocking operations
+};

package/package.json

@@ -1,11 +1,10 @@
 {
   "name": "@pioneer-platform/default-redis",
-  "version": "8.11.0",
+  "version": "8.11.2",
   "dependencies": {
     "@pioneer-platform/loggerdog": "^8.11.0",
     "dotenv": "^8.2.0",
-    "ioredis": "^4.17.3",
-    "ioredis-mock": "^4.21.1"
+    "redis": "^5.9.0"
   },
   "gitHead": "aeae28273014ab69b42f22abec159c6693a56c40"
 }