node-red-contrib-redis-variable 1.3.0 → 1.3.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "node-red-contrib-redis-variable",
-  "version": "1.3.0",
+  "version": "1.3.1",
   "description": "A comprehensive Node-RED node for Redis operations with universal payload-based configuration, automatic JSON handling, SSL/TLS support, and advanced pattern matching with pagination",
   "keywords": [
     "node-red",

package/redis-variable-config.js

@@ -157,6 +157,51 @@ module.exports = function (RED) {
             }
         };
 
+        // Check if configuration exists in context
+        this.hasValidConfiguration = function(msg, executingNode) {
+            try {
+                // If using string configuration for host and port, it's always valid
+                if (this.hostType === 'str' && this.portType === 'str') {
+                    return true;
+                }
+                
+                // For context-based configuration, check if the required values exist
+                let hasHost = false;
+                let hasPort = false;
+                
+                // Check host configuration
+                if (this.hostType === 'str') {
+                    hasHost = !!(this.host);
+                } else {
+                    const contextHost = this.parseCredentialValue(this.hostContext, this.hostType, msg, executingNode);
+                    hasHost = !!(contextHost);
+                }
+                
+                // Check port configuration
+                if (this.portType === 'str') {
+                    hasPort = !!(this.port);
+                } else {
+                    const contextPort = this.parseCredentialValue(this.portContext, this.portType, msg, executingNode);
+                    hasPort = !!(contextPort);
+                }
+                
+                // We need at least host and port to have a valid configuration
+                const result = hasHost && hasPort;
+                
+                if (executingNode && process.env.NODE_RED_DEBUG) {
+                    executingNode.log(`Configuration check - hasHost: ${hasHost}, hasPort: ${hasPort}, result: ${result}`);
+                }
+                
+                return result;
+                
+            } catch (error) {
+                if (executingNode) {
+                    executingNode.error(`Error checking configuration: ${error.message}`);
+                }
+                return false;
+            }
+        };
+
         // Get Redis connection options
         this.getConnectionOptions = function(msg, executingNode) {
             try {
@@ -301,6 +346,25 @@ module.exports = function (RED) {
                     return connections[id];
                 }
 
+                // Check if configuration is available before attempting connection
+                if (!this.hasValidConfiguration(msg, executingNode)) {
+                    if (executingNode) {
+                        // Only warn once per node to avoid spam
+                        if (!executingNode._configWarningShown) {
+                            executingNode.warn("Redis configuration not available in context. Skipping connection attempt.");
+                            executingNode._configWarningShown = true;
+                            
+                            // Reset warning flag after 30 seconds
+                            setTimeout(() => {
+                                if (executingNode) {
+                                    executingNode._configWarningShown = false;
+                                }
+                            }, 30000);
+                        }
+                    }
+                    return null;
+                }
+
                 const options = this.getConnectionOptions(msg, executingNode);
 
                 // Add connection limits to prevent infinite retry loops

package/redis-variable.js

@@ -280,7 +280,12 @@ module.exports = function (RED) {
                     if (!client) {
                         client = redisConfig.getClient(msg, node, node.id);
                         if (!client) {
-                            throw new Error("Failed to initialize Redis client");
+                            node.warn("Redis configuration not available. Operation skipped.");
+                            node.status({ fill: "yellow", shape: "ring", text: "no config" });
+                            msg.payload = { error: "Redis configuration not available" };
+                            send(msg);
+                            done();
+                            return;
                         }
                     }
                     
@@ -296,7 +301,12 @@ module.exports = function (RED) {
                                 client = null;
                                 client = redisConfig.getClient(msg, node, node.id);
                                 if (!client) {
-                                    throw new Error("Failed to recreate Redis client");
+                                    node.warn("Redis configuration not available during reconnection. Operation skipped.");
+                                    node.status({ fill: "yellow", shape: "ring", text: "no config" });
+                                    msg.payload = { error: "Redis configuration not available" };
+                                    send(msg);
+                                    done();
+                                    return;
                                 }
                             } catch (reconnectError) {
                                 throw new Error(`Failed to reconnect: ${reconnectError.message}`);