node-red-contrib-tcp-client-server-avd 1.0.0 → 1.1.0

package/README.md

@@ -26,10 +26,11 @@ npm install
 ### TCP Client Mode
 - Automatisch verbinden bij flow start (optioneel)
 - Handmatig verbinden/verbreken via messages
-- Automatische herconnectie met configureerbare instellingen
+- Robuuste automatische herconnectie (detectie via error, close en write-fouten)
+- Snelle herverbinding (standaard 2 sec, minimum 1 sec)
 - Ondersteuning voor verschillende dataformaten (String, Buffer, JSON, Hex)
 - Message delimiter ondersteuning voor message framing
-- Keep-alive optie
+- Keep-alive optie (aanbevolen voor detectie van half-open verbindingen)
 
 ### TCP Server Mode
 - Automatisch starten bij flow start (optioneel)
@@ -198,6 +199,15 @@ Als er geen topic is opgegeven, wordt de payload automatisch verzonden.
 }
 ```
 
+### ClientId bepalen (Server Mode)
+
+Om naar een specifieke client te sturen heb je de `clientId` nodig. Deze is beschikbaar in:
+
+1. **`client/connected`** – bij elke nieuwe clientverbinding
+2. **`data`** – bij elk ontvangen bericht (via `msg.clientId`)
+
+Bewaar de `clientId` uit een van deze messages en gebruik deze in je `send` message met `clientId: msg.clientId`.
+
 ## Configuratie Opties
 
 ### Client Mode
@@ -207,14 +217,14 @@ Als er geen topic is opgegeven, wordt de payload automatisch verzonden.
 - **IP Versie**: IPv4 of IPv6 (standaard: IPv4)
 - **Auto Verbinden**: Automatisch verbinden bij flow start (standaard: true)
 - **Herconnectie**: Automatische herconnectie bij verbroken verbinding (standaard: true)
-- **Reconnect Interval**: Tijd tussen reconnect pogingen in seconden (standaard: 5)
+- **Reconnect Interval**: Tijd tussen reconnect pogingen in seconden (standaard: 2, minimum: 1)
 - **Max Reconnect Pogingen**: Maximum aantal reconnect pogingen (0 = oneindig, standaard: 0)
-- **Connection Timeout**: Timeout voor connectiepogingen in seconden (standaard: 10)
+- **Connection Timeout**: Timeout voor het opzetten van de verbinding in seconden (standaard: 10). Wordt na succesvol verbinden uitgeschakeld; geen idle-timeout.
 - **Input Format**: Data formaat voor verzenden (String, Buffer, JSON, Hex String)
 - **Output Format**: Data formaat voor ontvangen (String, Buffer, JSON, Hex String)
 - **Encoding**: Character encoding voor String format (UTF-8, ASCII, Latin1, Base64)
 - **Message Delimiter**: Optionele delimiter voor message framing (bijv. `\n` of `\r\n`)
-- **Keep-Alive**: TCP keep-alive optie (standaard: false)
+- **Keep-Alive**: TCP keep-alive optie (standaard: false). Aanbevolen voor vroegtijdige detectie van verbroken verbindingen.
 
 ### Server Mode
 
@@ -228,7 +238,7 @@ Als er geen topic is opgegeven, wordt de payload automatisch verzonden.
 - **Output Format**: Data formaat voor ontvangen (String, Buffer, JSON, Hex String)
 - **Encoding**: Character encoding voor String format (UTF-8, ASCII, Latin1, Base64)
 - **Message Delimiter**: Optionele delimiter voor message framing (bijv. `\n` of `\r\n`)
-- **Keep-Alive**: TCP keep-alive optie (standaard: false)
+- **Keep-Alive**: TCP keep-alive optie (standaard: false). Aanbevolen voor vroegtijdige detectie van verbroken verbindingen.
 
 ## Status Weergave
 
@@ -286,7 +296,7 @@ Een TCP client met automatische herconnectie.
 - Port: 8080
 - Auto Verbinden: aan
 - Herconnectie: aan
-- Reconnect Interval: 5 seconden
+- Reconnect Interval: 2 seconden (of 1 voor snellere herverbinding)
 - Max Reconnect Pogingen: 0 (oneindig)
 
 ### Voorbeeld 4: Server met Meerdere Clients
@@ -300,10 +310,15 @@ Een TCP server die data ontvangt van meerdere clients en responses terugstuurt.
 
 **Function Node code:**
 ```javascript
-// Echo de data terug naar dezelfde client
-msg.topic = "send";
-msg.payload = "Echo: " + msg.payload;
-return msg;
+// Echo de data terug naar dezelfde client (clientId uit ontvangen bericht)
+if (msg.topic === "data") {
+    return {
+        topic: "send",
+        clientId: msg.clientId,
+        payload: "Echo: " + msg.payload
+    };
+}
+return null;
 ```
 
 ## Troubleshooting
@@ -341,6 +356,13 @@ MIT
 
 ## Versie Geschiedenis
 
+### Versie 1.1.0
+- Robuuste verbindingscontrole: error handler triggert socket.destroy() voor betrouwbare herverbinding
+- Write-error detectie: bij mislukte verzending wordt herverbinding gestart
+- Snellere herverbinding: standaard 2 sec, minimum 1 sec
+- Geen idle-timeout na verbinden: verbinding blijft open zolang beide partijen verbonden zijn
+- Voorkomen dubbele reconnect en race conditions
+
 ### Versie 1.0.0
 - Initiele release
 - TCP Client functionaliteit

package/nodes/tcp-client-server.html

@@ -11,7 +11,7 @@
             clientIpVersion: { value: "4" },
             clientAutoConnect: { value: true },
             clientReconnect: { value: true },
-            clientReconnectInterval: { value: 5, validate: function(v) { return /^\d+$/.test(v) && v >= 0; } },
+            clientReconnectInterval: { value: 2, validate: function(v) { return /^\d+$/.test(v) && v >= 1; } },
             clientMaxReconnectAttempts: { value: 0, validate: function(v) { return /^\d+$/.test(v) && v >= 0; } },
             clientConnectionTimeout: { value: 10, validate: function(v) { return /^\d+$/.test(v) && v > 0; } },
             clientInputFormat: { value: "string" },
@@ -118,7 +118,7 @@
         
         <div class="form-row">
             <label for="node-input-clientReconnectInterval"><i class="fa fa-clock-o"></i> Reconnect Interval (sec)</label>
-            <input type="number" id="node-input-clientReconnectInterval" placeholder="5" min="0">
+            <input type="number" id="node-input-clientReconnectInterval" placeholder="2" min="1" title="Minimum 1 seconde voor snelle herverbinding">
         </div>
         
         <div class="form-row">

package/nodes/tcp-client-server.js

@@ -118,7 +118,7 @@ module.exports = function(RED) {
             ipVersion: config.clientIpVersion || "4",
             autoConnect: config.clientAutoConnect !== false,
             reconnect: config.clientReconnect !== false,
-            reconnectInterval: parseInt(config.clientReconnectInterval) || 5,
+            reconnectInterval: Math.max(1, parseInt(config.clientReconnectInterval) || 2),
             maxReconnectAttempts: parseInt(config.clientMaxReconnectAttempts) || 0,
             connectionTimeout: parseInt(config.clientConnectionTimeout) || 10,
             inputFormat: config.clientInputFormat || "string",
@@ -257,6 +257,9 @@ module.exports = function(RED) {
 
             socket.on('close', function(hadError) {
                 node.clientConnected = false;
+                if (node.clientSocket === socket) {
+                    node.clientSocket = null;
+                }
                 node.status({ fill: "red", shape: "ring", text: "verbinding gesloten" });
                 
                 const msg = {
@@ -267,11 +270,15 @@ module.exports = function(RED) {
                 };
                 node.send(msg);
 
-                if (node.clientConfig.reconnect && (node.clientConfig.maxReconnectAttempts === 0 || node.clientReconnectAttempts < node.clientConfig.maxReconnectAttempts)) {
+                // Robuuste herverbinding: alleen starten als nog geen timer loopt
+                if (!node.clientReconnectTimer && node.clientConfig.reconnect && 
+                    (node.clientConfig.maxReconnectAttempts === 0 || node.clientReconnectAttempts < node.clientConfig.maxReconnectAttempts)) {
                     node.clientReconnectAttempts++;
+                    const intervalMs = node.clientConfig.reconnectInterval * 1000;
                     node.clientReconnectTimer = setTimeout(function() {
+                        node.clientReconnectTimer = null;
                         clientConnect();
-                    }, node.clientConfig.reconnectInterval * 1000);
+                    }, intervalMs);
                     node.status({ fill: "yellow", shape: "ring", text: `herconnectie in ${node.clientConfig.reconnectInterval}s...` });
                 }
             });
@@ -287,6 +294,8 @@ module.exports = function(RED) {
                     error: err.message
                 };
                 node.send(msg);
+                // Socket vernietigen zodat 'close' wordt getriggerd en herverbinding start
+                try { socket.destroy(); } catch (e) { /* negeer */ }
             });
 
             socket.on('timeout', function() {
@@ -330,9 +339,17 @@ module.exports = function(RED) {
                     toSend = Buffer.concat([formatted, delim]);
                 }
 
-                node.clientSocket.write(toSend);
+                node.clientSocket.write(toSend, function(err) {
+                    if (err) {
+                        node.clientConnected = false;
+                        node.error("Verzenden mislukt: " + err.message);
+                        try { node.clientSocket.destroy(); } catch (e) { /* negeer */ }
+                    }
+                });
             } catch (err) {
+                node.clientConnected = false;
                 node.error("Fout bij verzenden: " + err.message);
+                try { if (node.clientSocket) node.clientSocket.destroy(); } catch (e) { /* negeer */ }
             }
         }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "node-red-contrib-tcp-client-server-avd",
-  "version": "1.0.0",
+  "version": "1.1.0",
   "description": "Node-RED node voor TCP client en/of server functionaliteit",
   "keywords": [
     "node-red",
@@ -14,7 +14,7 @@
       "tcp-client-server-avd": "nodes/tcp-client-server.js"
     }
   },
-  "author": "",
+  "author": "Arie van Dijke",
   "license": "MIT",
   "engines": {
     "node": ">=12.0.0"