fix: Use correct client id pattern (#24)

Fixes #23

Author: tomquist

CHANGELOG.md

@@ -1,5 +1,8 @@
 # Changelog
 
+## [1.1.8]
+- Fix connection to remove broker by using the correct client ID pattern
+
 ## [1.1.7]
 - Added health check endpoint for better monitoring and automatic recovery
 - Added Docker health check configuration

package.json

@@ -1,6 +1,6 @@
 {
   "name": "hame-relay",
-  "version": "1.1.7",
+  "version": "1.1.8",
   "description": "MQTT message forwarder between brokers",
   "main": "dist/forwarder.js",
   "scripts": {

src/forwarder.ts

@@ -229,6 +229,7 @@ class MQTTForwarder {
   private initializeBrokers(): void {
     const options = {
       keepalive: 30,
+      clientId: this.generateClientId()
     };
     this.configBroker = mqtt.connect(this.config.broker_url, options);
 
@@ -243,6 +244,14 @@ class MQTTForwarder {
     this.setupBrokerEventHandlers();
   }
 
+  private generateClientId() {
+    let randomClientId = "";
+    for (let i = 0; i < 24; i++) {
+      randomClientId += Math.floor(Math.random() * 16).toString(16);
+    }
+    return `hm_${randomClientId}`;
+  }
+
   private setupBrokerEventHandlers(): void {
     // Config broker event handlers
     this.configBroker.on('connect', () => {