homebridge-midea-platform 0.4.2 → 0.4.4

package/homebridge-ui/public/index.html

@@ -187,6 +187,7 @@
       currentConfig.devices = currentConfig.devices || [];
       if (devices) {
         devices.forEach((device) => {
+          const validAuth = device.token && device.key;
           const tr = table.insertRow();
           const td = tr.insertCell();
           td.appendChild(document.createTextNode(device.name));
@@ -195,14 +196,21 @@
           tr.insertCell().appendChild(document.createTextNode(device.displayName));
           tr.insertCell().appendChild(document.createTextNode(device.model));
           tr.insertCell().appendChild(document.createTextNode(device.id));
-          tr.insertCell().appendChild(document.createTextNode(device.token.slice(0, 6) + '...' + device.token.slice(-4) +
-            '\n' + device.key.slice(0, 6) + '...' + device.key.slice(-4)
+          tr.insertCell().appendChild(document.createTextNode(
+            device.token ? 
+              device.token.slice(0, 6) + '...' + device.token.slice(-4) :
+              'token missing' + '\n' + 
+            device.key ? 
+              device.key.slice(0, 6) + '...' + device.key.slice(-4) :
+              'key missing'
           ));
           tr.insertCell().appendChild(document.createTextNode(device.ip));
 
           const addCell = tr.insertCell();
           if (device.displayName === 'Unknown') {
             addCell.appendChild(document.createTextNode('Not supported'));
+          } else if (!validAuth) {
+            addCell.appendChild(document.createTextNode('Invalid credentials'));
           } else if (currentDevice = currentConfig.devices.find((d) => d.id === device.id)) {
             // addCell.appendChild(document.createTextNode('Already added'));
             debugLog(`Existing device:\n${JSON.stringify(currentDevice, null, 2)}`);

package/homebridge-ui/server.js

@@ -75,11 +75,11 @@ class UiServer extends HomebridgePluginUiServer {
     super();
     // Obtain the plugin configuration from homebridge config JSON file.
     const config = require(this.homebridgeConfigPath).platforms.find((obj) => obj.platform === 'midea-platform');
-    this.logger = new Logger(config.uiDebug ? config.uiDebug : false);
+    this.logger = new Logger(config?.uiDebug ? config.uiDebug : false);
     this.logger.info(`Custom UI created.`);
     this.logger.debug(`ENV:\n${JSON.stringify(process.env, null, 2)}`);
     this.security = new LocalSecurity();
-    this.promiseSocket = new PromiseSocket(this.logger, config.logRecoverableErrors ? config.logRecoverableErrors : false);
+    this.promiseSocket = new PromiseSocket(this.logger, config?.logRecoverableErrors ? config.logRecoverableErrors : false);
 
     this.onRequest('/login', async ({ username, password, registeredApp }) => {
       try {
@@ -112,6 +112,9 @@ class UiServer extends HomebridgePluginUiServer {
     this.onRequest('/discover', async () => {
       try {
         const devices = await this.blockingDiscover();
+        for (const device of devices) {
+          await this.getNewCredentials(device);
+        }
         this.logger.debug(`All devices:\n${JSON.stringify(devices, null, 2)}`);
         return devices
           .filter((a) => Object.keys(a).length > 0)
@@ -155,10 +158,7 @@ class UiServer extends HomebridgePluginUiServer {
       }
       i++;
     }
-    if (!device.token || !device.key) {
-      throw new Error(`[${device.name}] Authentication failed, token/key undefined.`);
-    }
-    this.logger.debug(`Token: ${device.token}, Key: ${device.key}`);
+    this.logger.debug(`[${device.name}] Token: ${device.token}, Key: ${device.key}`);
     return;
   }
 
@@ -203,7 +203,6 @@ class UiServer extends HomebridgePluginUiServer {
       this.logger.info('Start device discovery...');
       discover.startDiscover();
       discover.on('device', async (device) => {
-        await this.getNewCredentials(device);
         switch (device.type) {
           case DeviceType.AIR_CONDITIONER:
             device['displayName'] = 'Air Conditioner';

package/package.json

@@ -1,7 +1,7 @@
 {
-  "displayName": "Homebridge Midea",
+  "displayName": "Homebridge Midea Platform",
   "name": "homebridge-midea-platform",
-  "version": "0.4.2",
+  "version": "0.4.4",
   "description": "Homebridge plugin for Midea devices",
   "license": "Apache-2.0",
   "repository": {
@@ -12,7 +12,7 @@
     "url": "https://github.com/kovapatrik/homebridge-midea/issues"
   },
   "engines": {
-    "node": "^18.17.0",
+    "node": "^18.17.0 || ^20.9.0",
     "homebridge": "^1.6.0"
   },
   "main": "dist/index.js",