homebridge-smartsystem 6.8.5 → 6.8.8

package/README.md

@@ -302,13 +302,16 @@ Homebridge UI version
 - 4: better up-down values to and from HB
 
 
-### v6.8.0 - 08/07/2025
+### v6.8.x - 08/07/2025
 - 0: first version including the proxy
 - 1: max/min values for hbValues
 - 2: HB up/down -> don't respond to motor unit's, only macro
 - 3: remember request before the login kicked in (and redirect to that one, once the login was succesful)
 - 4: now using only TCP sockets to the server
 - 5: making it more robust / inline with repo-service version
+- 6: 26 aug / removed double error handlers on cloudSocket
+- 7: 28 aug / restart if no free connections
+- 8: 28 aug / less logging (through logConfig) for device set/get functions
 
 ## Hardware 
 A Raspberry Pi that connects to a Duotecno IP Node 

package/accessories/accessory.js

@@ -86,7 +86,7 @@ class Accessory {
     }
     updateState() {
         this.me.getCharacteristic(this.homebridge.Characteristic.On).updateValue(this.unit.status);
-        // log("accessory", "Received status change -> update accessory -> " + this.unit.getName() + " -> on = " + this.unit.status);
+        (0, logger_1.log)("accessory", "Received status change -> update accessory -> " + this.unit.getName() + " -> on = " + this.unit.status);
     }
 }
 exports.Accessory = Accessory;

package/accessories/accessory.ts

@@ -107,7 +107,7 @@ export class Accessory {
 
   updateState() {
     this.me.getCharacteristic(this.homebridge.Characteristic.On).updateValue(this.unit.status);
-    // log("accessory", "Received status change -> update accessory -> " + this.unit.getName() + " -> on = " + this.unit.status);
+    log("accessory", "Received status change -> update accessory -> " + this.unit.getName() + " -> on = " + this.unit.status);
   }
 
 }

package/accessories/dimmer.js

@@ -69,7 +69,7 @@ class Dimmer extends accessory_1.Accessory {
     }
     updateState() {
         this.me.getCharacteristic(this.homebridge.Characteristic.Brightness).updateValue(this.unit.value);
-        // log("accessory", "Received status change -> update accessory -> " + this.unit.getName() + " -> Brightness = " + this.unit.value);
+        (0, logger_1.log)("accessory", "Received status change -> update accessory -> " + this.unit.getName() + " -> Brightness = " + this.unit.value);
     }
 }
 exports.Dimmer = Dimmer;

package/accessories/dimmer.ts

@@ -84,7 +84,7 @@ export class Dimmer extends Accessory {
 
   updateState() {
     this.me.getCharacteristic(this.homebridge.Characteristic.Brightness).updateValue(this.unit.value);
-    // log("accessory", "Received status change -> update accessory -> " + this.unit.getName() + " -> Brightness = " + this.unit.value);
+    log("accessory", "Received status change -> update accessory -> " + this.unit.getName() + " -> Brightness = " + this.unit.value);
   }
 
 }

package/accessories/garagedoor.js

@@ -87,7 +87,7 @@ class GarageDoor extends accessory_1.Accessory {
     // in response to Duotecno status messages
     updateState() {
         const value = this.DT2HB(this.unit.status);
-        // log("accessory", "Received updateState -> Homekit GarageDoor for " + this.unit.node.getName() + " - " + this.unit.getName() + " -> " + this.unit.status + " / " + this.unit.value + " -> passing to HB: " + value);
+        (0, logger_1.log)("accessory", "Received updateState -> Homekit GarageDoor for " + this.unit.node.getName() + " - " + this.unit.getName() + " -> " + this.unit.status + " / " + this.unit.value + " -> passing to HB: " + value);
         this.me.getCharacteristic(this.homebridge.Characteristic.CurrentDoorState).updateValue(value);
     }
 }

package/accessories/garagedoor.ts

@@ -117,7 +117,7 @@ export class GarageDoor extends Accessory {
   // in response to Duotecno status messages
   updateState() {
     const value = this.DT2HB(this.unit.status);
-    // log("accessory", "Received updateState -> Homekit GarageDoor for " + this.unit.node.getName() + " - " + this.unit.getName() + " -> " + this.unit.status + " / " + this.unit.value + " -> passing to HB: " + value);
+    log("accessory", "Received updateState -> Homekit GarageDoor for " + this.unit.node.getName() + " - " + this.unit.getName() + " -> " + this.unit.status + " / " + this.unit.value + " -> passing to HB: " + value);
     this.me.getCharacteristic(this.homebridge.Characteristic.CurrentDoorState).updateValue(value);
   }
 

package/accessories/lock.js

@@ -69,7 +69,7 @@ class Lock extends accessory_1.Accessory {
     }
     // in response to Duotecno status messages
     updateState() {
-        // log("accessory", "Received updateState -> Homekit Lock for " + this.unit.node.getName() + " - " + this.unit.getName() + " -> " + this.unit.status);
+        (0, logger_1.log)("accessory", "Received updateState -> Homekit Lock for " + this.unit.node.getName() + " - " + this.unit.getName() + " -> " + this.unit.status);
         this.me.getCharacteristic(this.homebridge.Characteristic.LockCurrentState).updateValue(this.unit.status);
     }
 }

package/accessories/lock.ts

@@ -89,7 +89,7 @@ export class Lock extends Accessory {
 
   // in response to Duotecno status messages
   updateState() {
-    // log("accessory", "Received updateState -> Homekit Lock for " + this.unit.node.getName() + " - " + this.unit.getName() + " -> " + this.unit.status);
+    log("accessory", "Received updateState -> Homekit Lock for " + this.unit.node.getName() + " - " + this.unit.getName() + " -> " + this.unit.status);
     this.me.getCharacteristic(this.homebridge.Characteristic.LockCurrentState).updateValue(this.unit.status);
   }
 

package/accessories/mood.js

@@ -51,7 +51,7 @@ class Mood extends accessory_1.Accessory {
     }
     updateState() {
         this.me.getCharacteristic(this.homebridge.Characteristic.On).updateValue(!!this.unit.value);
-        // log("accessory", "Received status change -> update accessory -> " + this.unit.getName() + " -> On = " + !!this.unit.value);
+        (0, logger_1.log)("accessory", "Received status change -> update accessory -> " + this.unit.getName() + " -> On = " + !!this.unit.value);
     }
 }
 exports.Mood = Mood;

package/accessories/mood.ts

@@ -62,7 +62,7 @@ export class Mood extends Accessory {
   }
   updateState() {
     this.me.getCharacteristic(this.homebridge.Characteristic.On).updateValue(!!this.unit.value);
-    // log("accessory", "Received status change -> update accessory -> " + this.unit.getName() + " -> On = " + !!this.unit.value);
+    log("accessory", "Received status change -> update accessory -> " + this.unit.getName() + " -> On = " + !!this.unit.value);
   }
 
 }

package/accessories/temperature.js

@@ -34,7 +34,7 @@ class Temperature extends accessory_1.Accessory {
     }
     updateState() {
         this.me.getCharacteristic(this.homebridge.Characteristic.CurrentTemperature).updateValue(this.unit.value / 10.0);
-        // log("accessory", "Received updateState -> Homekit Temperature -> " + this.unit.getName() + " -> temp = " +(<number>this.unit.value) / 10.0);
+        (0, logger_1.log)("accessory", "Received updateState -> Homekit Temperature -> " + this.unit.getName() + " -> temp = " + this.unit.value / 10.0);
     }
 }
 exports.Temperature = Temperature;

package/accessories/temperature.ts

@@ -42,7 +42,7 @@ export class Temperature extends Accessory {
 
   updateState() {
     this.me.getCharacteristic(this.homebridge.Characteristic.CurrentTemperature).updateValue((<number>this.unit.value) / 10.0);
-    // log("accessory", "Received updateState -> Homekit Temperature -> " + this.unit.getName() + " -> temp = " +(<number>this.unit.value) / 10.0);
+    log("accessory", "Received updateState -> Homekit Temperature -> " + this.unit.getName() + " -> temp = " +(<number>this.unit.value) / 10.0);
   }
 
 }

package/accessories/windowcovering.js

@@ -134,7 +134,7 @@ class WindowCovering extends accessory_1.Accessory {
     // called in response to Duotecno status update
     updateState() {
         const value = this.DT2HB(this.unit.status);
-        // log("accessory", "Received updateState -> Homekit WindowCovering for " + this.unit.node.getName() + " - " + this.unit.getName() + " -> " + this.unit.status + " / " + this.unit.value + " -> passing: " + value);
+        (0, logger_1.log)("accessory", "Received updateState -> Homekit WindowCovering for " + this.unit.node.getName() + " - " + this.unit.getName() + " -> " + this.unit.status + " / " + this.unit.value + " -> passing: " + value);
         this.me.getCharacteristic(this.homebridge.Characteristic.CurrentPosition).updateValue(value);
         if (this.unit.status == types_1.UnitState.kClosing) {
             this.me.getCharacteristic(this.homebridge.Characteristic.TargetPosition).updateValue(0);

package/accessories/windowcovering.ts

@@ -161,7 +161,7 @@ export class WindowCovering extends Accessory {
   // called in response to Duotecno status update
   updateState() {
     const value = this.DT2HB(this.unit.status);
-    // log("accessory", "Received updateState -> Homekit WindowCovering for " + this.unit.node.getName() + " - " + this.unit.getName() + " -> " + this.unit.status + " / " + this.unit.value + " -> passing: " + value);
+    log("accessory", "Received updateState -> Homekit WindowCovering for " + this.unit.node.getName() + " - " + this.unit.getName() + " -> " + this.unit.status + " / " + this.unit.value + " -> passing: " + value);
 
     this.me.getCharacteristic(this.homebridge.Characteristic.CurrentPosition).updateValue(value);
 

package/duotecno/logger.js

@@ -16,6 +16,7 @@ exports.logSettings = {
     "*": LogLevel.log,
     "master": LogLevel.log,
     "protocol": LogLevel.noLog,
+    "accessory": LogLevel.noLog,
     "webapp": LogLevel.log,
     "p1": LogLevel.log,
     "shelly": LogLevel.log,

package/duotecno/logger.ts

@@ -11,6 +11,7 @@ export const logSettings = {
   "*": LogLevel.log,
   "master": LogLevel.log,
   "protocol": LogLevel.noLog,
+  "accessory": LogLevel.noLog,
   "webapp": LogLevel.log,
   "p1": LogLevel.log,
   "shelly": LogLevel.log,

package/duotecno/types.ts

@@ -17,7 +17,7 @@ export enum NodeType { kNoNode = 0, kStandardNode = 1, kGatewayNode = 4, kModemN
 
 // States
 export enum UnitState { kOpening = 4, kClosing = 3, kOpen = 2, kClosed = 1, kStopped = 0 };
-export enum UnitMotorCmd { kClose = 5, kOpen = 4, kStop = 3};
+export enum UnitMotorCmd { kClose = 5, kOpen = 4, kStop = 3 };
 
 export enum UnitType { 
   kNoType = 0, kDimmer = 1, kSwitch = 2, kInput = 3, 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "homebridge-smartsystem",
   "displayname": "Duotecno Bridge",
-  "version": "6.8.5",
+  "version": "6.8.8",
   "description": "SmartServer (Proxy TCP sockets to the cloud, Smappee MQTT, Duotecno IP Nodes, Homekit interface)",
   "main": "index.js",
   "author": "Johan Coppieters",

package/server/proxy.js

@@ -38,6 +38,7 @@ catch (e) {
     setProxyConfig();
     console.log("No config-proxy.json found, using default values from homebridge or other.");
 }
+let connectionChecker = null;
 const gCloudConnections = [];
 const kDebug = config.debug || false; // enable debug mode from config
 /////////////
@@ -60,13 +61,23 @@ function error(str) {
 ///////////////////////////////
 // Start up the proxy server //
 ///////////////////////////////
-if (config.uniqueId) {
-    makeNewCloudConnection(config.masterAddress, config.masterPort, config.cloudServer, config.cloudPort, config.uniqueId);
-}
+cleanStart(true);
 function makeNewCloudConnection(master, masterPort, server, serverPort, uniqueId, count = 0) {
+    // retry making a cloud connection
+    function reconnect(err) {
+        error(`[PROXY] → [CLOUD] Failed to connect ${count + 1} times to the Cloud server: ${err.message}`);
+        if (count < 3) {
+            setTimeout(() => {
+                makeNewCloudConnection(master, masterPort, server, serverPort, uniqueId, count + 1);
+            }, 1000);
+        }
+    }
     const cloudSocket = new net.Socket();
+    cloudSocket.once('error', reconnect);
+    log(`[PROXY] → [CLOUD] Attempt ${count + 1} to start a new free connection for ${config.uniqueId} to the Cloud at ${server}:${serverPort}`);
     cloudSocket.connect(serverPort, server, () => {
         log(`[PROXY] → [CLOUD] Connected to Cloud at ${server}:${serverPort}`);
+        cloudSocket.removeListener('error', reconnect);
         // Send unique ID as the first message to identify this proxy/device
         cloudSocket.write(`[${uniqueId}]`);
         debug(`[PROXY] → [CLOUD] Sent unique ID: ${uniqueId}`);
@@ -74,27 +85,35 @@ function makeNewCloudConnection(master, masterPort, server, serverPort, uniqueId
         gCloudConnections.push(context);
         log(`[PROXY] → [CLOUD] New free connection #${gCloudConnections.length} to the Cloud at ${server}:${serverPort}`);
     });
-    cloudSocket.on('error', (err) => {
-        error(`[PROXY] → [CLOUD] Failed to connect ${count + 1} times to the Cloud server: ${err.message}`);
-        if (count < 3) {
-            setTimeout(() => {
-                makeNewCloudConnection(master, masterPort, server, serverPort, uniqueId, count + 1);
-            }, 1000);
-        }
-    });
-    log(`[PROXY] → [CLOUD] Attempt ${count + 1} to start a new free connection for ${config.uniqueId} to the Cloud at ${server}:${serverPort}`);
 }
 exports.makeNewCloudConnection = makeNewCloudConnection;
 ////////////////
 // Restarting //
 ////////////////
 function cleanStart(restart = false) {
-    gCloudConnections.forEach(ctx => ctx.cleanupSockets());
-    log(`[PROXY] → [CLOUD] Cleaned up all ${gCloudConnections.length} cloud connections.`);
-    gCloudConnections.splice(0, gCloudConnections.length); // clear the array
+    if (gCloudConnections.length) {
+        gCloudConnections.forEach(ctx => ctx.cleanupSockets());
+        log(`[PROXY] → [CLOUD] Cleaned up all ${gCloudConnections.length} cloud connections.`);
+        gCloudConnections.splice(0, gCloudConnections.length); // clear the array
+    }
+    if (connectionChecker) {
+        clearInterval(connectionChecker);
+        connectionChecker = null;
+    }
     if (restart) {
         if (config.uniqueId) {
             makeNewCloudConnection(config.masterAddress, config.masterPort, config.cloudServer, config.cloudPort, config.uniqueId);
+            // check every 5 second for a free connection
+            connectionChecker = setInterval(() => {
+                const freeConnection = gCloudConnections.find((ctx) => (!ctx.deviceSocket) && ctx.cloudSocket && (ctx.cloudSocket.readyState === 'open'));
+                if (freeConnection) {
+                    log(`[PROXY] → [CLOUD] Found free connection #${freeConnection.uniqueId} to the Cloud -> we're still ok.`);
+                }
+                else {
+                    error(`[PROXY] → [CLOUD] No free connections available, restarting proxy. ***************************`);
+                    cleanStart(true);
+                }
+            }, 5000);
         }
         else {
             log(`[PROXY] → [CLOUD] Not restarting, no unique ID configured, not starting the proxy.`);
@@ -293,6 +312,8 @@ function restart() {
 // Graceful shutdown handler
 function handleShutdown(signal) {
     log(`[PROXY] → [SYSTEM] Received ${signal}, shutting down gracefully...`);
+    if (connectionChecker)
+        clearInterval(connectionChecker);
     // Close all cloud connections
     gCloudConnections.forEach(ctx => ctx.cleanupSockets());
     gCloudConnections.splice(0, gCloudConnections.length);

package/server/proxy.ts

@@ -51,6 +51,7 @@ try {
 }
 
 
+let connectionChecker: NodeJS.Timer | null = null;
 
 const gCloudConnections: Array<Context> = [];
 const kDebug = config.debug || false; // enable debug mode from config
@@ -77,16 +78,28 @@ function error(str: string) {
 ///////////////////////////////
 // Start up the proxy server //
 ///////////////////////////////
-if (config.uniqueId) {
-  makeNewCloudConnection(config.masterAddress, config.masterPort, config.cloudServer, config.cloudPort, config.uniqueId);
-}
+cleanStart(true);
 
 
 export function makeNewCloudConnection(master: string, masterPort: number, server: string, serverPort: number, uniqueId: string, count = 0) {
+  // retry making a cloud connection
+  function reconnect(err: Error) {
+    error(`[PROXY] → [CLOUD] Failed to connect ${count+1} times to the Cloud server: ${err.message}`);
+    if (count < 3) {
+      setTimeout(() => {
+        makeNewCloudConnection(master, masterPort, server, serverPort, uniqueId, count + 1);
+      }, 1000);
+    }
+  }
+
   const cloudSocket = new net.Socket();
-  
+  cloudSocket.once('error', reconnect);
+
+  log(`[PROXY] → [CLOUD] Attempt ${count+1} to start a new free connection for ${config.uniqueId} to the Cloud at ${server}:${serverPort}`);
+
   cloudSocket.connect(serverPort, server, () => {
     log(`[PROXY] → [CLOUD] Connected to Cloud at ${server}:${serverPort}`);
+    cloudSocket.removeListener('error', reconnect);
     
     // Send unique ID as the first message to identify this proxy/device
     cloudSocket.write(`[${uniqueId}]`);
@@ -96,17 +109,6 @@ export function makeNewCloudConnection(master: string, masterPort: number, serve
     gCloudConnections.push(context);
     log(`[PROXY] → [CLOUD] New free connection #${gCloudConnections.length} to the Cloud at ${server}:${serverPort}`);
   });
-
-  cloudSocket.on('error', (err) => {
-    error(`[PROXY] → [CLOUD] Failed to connect ${count+1} times to the Cloud server: ${err.message}`);
-    if (count < 3) {
-      setTimeout(() => {
-        makeNewCloudConnection(master, masterPort, server, serverPort, uniqueId, count + 1);
-      }, 1000);
-    }
-  });
-
-  log(`[PROXY] → [CLOUD] Attempt ${count+1} to start a new free connection for ${config.uniqueId} to the Cloud at ${server}:${serverPort}`);
 }
 
 
@@ -114,14 +116,35 @@ export function makeNewCloudConnection(master: string, masterPort: number, serve
 // Restarting //
 ////////////////
 export function cleanStart(restart: boolean = false) {
-  gCloudConnections.forEach(ctx => ctx.cleanupSockets());
-  log(`[PROXY] → [CLOUD] Cleaned up all ${gCloudConnections.length} cloud connections.`);
+  if (gCloudConnections.length) {
+    gCloudConnections.forEach(ctx => ctx.cleanupSockets());
+    log(`[PROXY] → [CLOUD] Cleaned up all ${gCloudConnections.length} cloud connections.`);
 
-  gCloudConnections.splice(0, gCloudConnections.length); // clear the array
+    gCloudConnections.splice(0, gCloudConnections.length); // clear the array
+  }
+
+  if (connectionChecker) {
+    clearInterval(connectionChecker);
+    connectionChecker = null;
+  }
 
   if (restart) {
     if (config.uniqueId) {
       makeNewCloudConnection(config.masterAddress, config.masterPort, config.cloudServer, config.cloudPort, config.uniqueId);
+
+      // check every 5 second for a free connection
+      connectionChecker = setInterval(() => {
+        const freeConnection = gCloudConnections.find((ctx: Context) =>
+          (!ctx.deviceSocket) && ctx.cloudSocket && (ctx.cloudSocket.readyState === 'open')
+        );
+        if (freeConnection) {
+          log(`[PROXY] → [CLOUD] Found free connection #${freeConnection.uniqueId} to the Cloud -> we're still ok.`);
+        } else {
+          error(`[PROXY] → [CLOUD] No free connections available, restarting proxy. ***************************`);
+          cleanStart(true);
+        }
+      }, 5000);
+
     } else {
       log(`[PROXY] → [CLOUD] Not restarting, no unique ID configured, not starting the proxy.`);
     }
@@ -364,6 +387,8 @@ function restart() {
 function handleShutdown(signal: string) {
   log(`[PROXY] → [SYSTEM] Received ${signal}, shutting down gracefully...`);
 
+  if (connectionChecker)clearInterval(connectionChecker);
+
   // Close all cloud connections
   gCloudConnections.forEach(ctx => ctx.cleanupSockets());
   gCloudConnections.splice(0, gCloudConnections.length);