iobroker.js-controller 4.0.18 → 4.0.21

package/conf/sources-dist.json

@@ -597,7 +597,7 @@
     "meta": "https://raw.githubusercontent.com/PLCHome/ioBroker.growatt/master/io-package.json",
     "icon": "https://raw.githubusercontent.com/PLCHome/ioBroker.growatt/master/admin/growatt.png",
     "type": "energy",
-    "version": "1.1.7"
+    "version": "1.1.8"
   },
   "gruenbeck": {
     "meta": "https://raw.githubusercontent.com/TA2k/ioBroker.gruenbeck/master/io-package.json",
@@ -831,7 +831,7 @@
     "meta": "https://raw.githubusercontent.com/ioBroker/ioBroker.influxdb/master/io-package.json",
     "icon": "https://raw.githubusercontent.com/ioBroker/ioBroker.influxdb/master/admin/influxdb.png",
     "type": "storage",
-    "version": "1.9.5"
+    "version": "2.6.3"
   },
   "info": {
     "meta": "https://raw.githubusercontent.com/iobroker-community-adapters/ioBroker.info/master/io-package.json",
@@ -890,7 +890,7 @@
   "js-controller": {
     "meta": "https://raw.githubusercontent.com/ioBroker/ioBroker.js-controller/master/packages/controller/io-package.json",
     "type": "general",
-    "version": "4.0.15"
+    "version": "4.0.19"
   },
   "kecontact": {
     "meta": "https://raw.githubusercontent.com/iobroker-community-adapters/ioBroker.kecontact/master/io-package.json",
@@ -932,7 +932,7 @@
     "meta": "https://raw.githubusercontent.com/hombach/ioBroker.kostal-piko-ba/master/io-package.json",
     "icon": "https://raw.githubusercontent.com/hombach/ioBroker.kostal-piko-ba/master/admin/picoba.png",
     "type": "energy",
-    "version": "1.2.1"
+    "version": "1.4.2"
   },
   "lametric": {
     "meta": "https://raw.githubusercontent.com/klein0r/ioBroker.lametric/master/io-package.json",
@@ -1064,7 +1064,7 @@
     "meta": "https://raw.githubusercontent.com/iobroker-community-adapters/ioBroker.luxtronik1/master/io-package.json",
     "icon": "https://raw.githubusercontent.com/iobroker-community-adapters/ioBroker.luxtronik1/master/admin/luxtronik1.png",
     "type": "climate-control",
-    "version": "0.3.4"
+    "version": "0.3.5"
   },
   "material": {
     "meta": "https://raw.githubusercontent.com/ioBroker/ioBroker.material/master/io-package.json",
@@ -1298,7 +1298,7 @@
     "meta": "https://raw.githubusercontent.com/Bart1909/ioBroker.netatmo-crawler/master/io-package.json",
     "icon": "https://raw.githubusercontent.com/Bart1909/ioBroker.netatmo-crawler/master/admin/netatmo-crawler.png",
     "type": "weather",
-    "version": "0.7.1"
+    "version": "0.8.1"
   },
   "netatmo-energy": {
     "meta": "https://raw.githubusercontent.com/Homemade-Disaster/ioBroker.netatmo-energy/master/io-package.json",
@@ -1400,7 +1400,7 @@
     "meta": "https://raw.githubusercontent.com/iobroker-community-adapters/ioBroker.openknx/master/io-package.json",
     "icon": "https://raw.githubusercontent.com/iobroker-community-adapters/ioBroker.openknx/master/admin/openknx.png",
     "type": "iot-systems",
-    "version": "0.1.15"
+    "version": "0.1.22"
   },
   "openligadb": {
     "meta": "https://raw.githubusercontent.com/oweitman/ioBroker.openligadb/master/io-package.json",
@@ -1706,7 +1706,7 @@
     "meta": "https://raw.githubusercontent.com/iobroker-community-adapters/ioBroker.shelly/master/io-package.json",
     "icon": "https://raw.githubusercontent.com/iobroker-community-adapters/ioBroker.shelly/master/admin/shelly.png",
     "type": "iot-systems",
-    "version": "4.0.7"
+    "version": "5.3.2"
   },
   "shuttercontrol": {
     "meta": "https://raw.githubusercontent.com/simatec/ioBroker.shuttercontrol/master/io-package.json",
@@ -1802,7 +1802,7 @@
     "meta": "https://raw.githubusercontent.com/afuerhoff/ioBroker.solarviewdatareader/master/io-package.json",
     "icon": "https://raw.githubusercontent.com/afuerhoff/ioBroker.solarviewdatareader/master/admin/solarviewdatareader.png",
     "type": "energy",
-    "version": "1.0.3"
+    "version": "1.0.5"
   },
   "solarwetter": {
     "meta": "https://raw.githubusercontent.com/Pix---/ioBroker.solarwetter/master/io-package.json",
@@ -2243,8 +2243,8 @@
     "version": "0.1.2"
   },
   "weatherunderground": {
-    "meta": "https://raw.githubusercontent.com/dschaedl/ioBroker.weatherunderground/master/io-package.json",
-    "icon": "https://raw.githubusercontent.com/dschaedl/ioBroker.weatherunderground/master/admin/wu.png",
+    "meta": "https://raw.githubusercontent.com/iobroker-community-adapters/ioBroker.weatherunderground/master/io-package.json",
+    "icon": "https://raw.githubusercontent.com/iobroker-community-adapters/ioBroker.weatherunderground/master/admin/wu.png",
     "type": "weather",
     "version": "3.3.1"
   },
@@ -2336,7 +2336,7 @@
     "meta": "https://raw.githubusercontent.com/foxriver76/ioBroker.xbox/master/io-package.json",
     "icon": "https://raw.githubusercontent.com/foxriver76/ioBroker.xbox/master/admin/xbox.png",
     "type": "multimedia",
-    "version": "0.7.7"
+    "version": "0.7.8"
   },
   "xs1": {
     "meta": "https://raw.githubusercontent.com/frankjoke/ioBroker.xs1/master/io-package.json",
@@ -2372,7 +2372,7 @@
     "meta": "https://raw.githubusercontent.com/ioBroker/ioBroker.yr/master/io-package.json",
     "icon": "https://raw.githubusercontent.com/ioBroker/ioBroker.yr/master/admin/yr.png",
     "type": "weather",
-    "version": "5.0.0"
+    "version": "5.1.2"
   },
   "zehnder-cloud": {
     "meta": "https://raw.githubusercontent.com/TA2k/ioBroker.zehnder-cloud/master/io-package.json",
@@ -2398,4 +2398,4 @@
     "type": "hardware",
     "version": "2.4.3"
   }
-}
+}

package/io-package.json

@@ -1,7 +1,7 @@
 {
   "common": {
     "name": "js-controller",
-    "version": "4.0.18",
+    "version": "4.0.21",
     "platform": "Javascript/Node.js",
     "controller": true,
     "mode": "daemon",
@@ -19,19 +19,31 @@
       "zh-cn": "JS控制器"
     },
     "news": {
-      "4.0.18": {
-        "en": "Also stop instances before GitHub installs\nFix eraseOnUpload logic\nFix invalid backup period warning\nFix maxSize property for logging in dist file\nPrevent some crash cases reported by Sentry",
-        "de": "Stoppen Sie außerdem Instanzen, bevor GitHub installiert wird\nEraseOnUpload-Logik korrigiert\nWarnung zu ungültigem Sicherungszeitraum behoben\nKorrigieren Sie die maxSize-Eigenschaft für die Anmeldung in der Dist-Datei\nVerhindern Sie einige von Sentry gemeldete Absturzfälle",
-        "ru": "Также останавливайте экземпляры перед установкой GitHub.\nИсправление логики EraseOnUpload\nИсправить предупреждение о недопустимом периоде резервного копирования\nИсправлено свойство maxSize для входа в файл dist.\nПредотвращение некоторых случаев сбоев, о которых сообщает Sentry.",
-        "pt": "Também interrompa instâncias antes da instalação do GitHub\nCorrigir a lógica eraseOnUpload\nCorrigir aviso de período de backup inválido\nCorrige a propriedade maxSize para fazer login no arquivo dist\nPrevenir alguns casos de travamento relatados pelo Sentinela",
-        "nl": "Stop ook instanties voordat GitHub wordt geïnstalleerd\nEraseOnUpload-logica repareren\nWaarschuwing voor ongeldige back-upperiode repareren\nFix maxSize-eigenschap voor inloggen in dist-bestand\nVoorkom enkele door Sentry gemelde crashgevallen",
-        "fr": "Arrêtez également les instances avant l'installation de GitHub\nCorrection de la logique eraseOnUpload\nCorrection de l'avertissement de période de sauvegarde non valide\nCorrection de la propriété maxSize pour la connexion au fichier dist\nEmpêcher certains cas de plantage signalés par Sentry",
-        "it": "Arresta anche le istanze prima dell'installazione di GitHub\nRisolto il problema con la logica di cancellazione durante il caricamento\nCorrezione dell'avviso di periodo di backup non valido\nCorrezione della proprietà maxSize per l'accesso al file dist\nPrevenire alcuni casi di crash segnalati da Sentry",
-        "es": "También detenga las instancias antes de que se instale GitHub\nArreglar la lógica eraseOnUpload\nArreglar la advertencia de período de copia de seguridad no válido\nArreglar la propiedad maxSize para iniciar sesión en el archivo dist\nEvitar algunos casos de bloqueo informados por Sentry",
-        "pl": "Zatrzymaj również instancje przed instalacją GitHub\nNapraw logikę eraseOnUpload\nNapraw ostrzeżenie o nieprawidłowym okresie tworzenia kopii zapasowej\nNapraw właściwość maxSize do logowania w pliku dist\nZapobiegaj niektórym przypadkom awarii zgłaszanych przez Sentry",
-        "zh-cn": "在 GitHub 安装之前停止实例\n修复 eraseOnUpload 逻辑\n修复无效备份周期警告\n修复用于登录 dist 文件的 maxSize 属性\n防止 Sentry 报告的一些崩溃案例"
+      "4.0.21": {
+        "en": "Fix Backup Restore crash case",
+        "de": "Beheben Sie den Absturzfall der Sicherungswiederherstellung",
+        "ru": "Исправить случай сбоя восстановления резервной копии",
+        "pt": "Corrigir caso de falha de restauração de backup",
+        "nl": "Fix Backup Herstel crashgeval",
+        "fr": "Correction du cas de plantage de restauration de sauvegarde",
+        "it": "Risolto il caso di arresto anomalo del ripristino del backup",
+        "es": "Solucionar caso de bloqueo de restauración de copia de seguridad",
+        "pl": "Napraw przypadek awarii przywracania kopii zapasowej",
+        "zh-cn": "修复备份还原崩溃案例"
       },
-      "4.0.15": {
+      "4.0.20": {
+        "en": "Add missing axios dependency\nOnly log an error if an error is happening when activating/deactivating adapter via .alive states\nFix edge case in redis simulator MULTI/EXEC logic\nPrevent some crash cases reported by Sentry",
+        "de": "Fehlende Axios-Abhängigkeit hinzugefügt\nProtokollieren Sie nur dann einen Fehler, wenn beim Aktivieren/Deaktivieren des Adapters über .alive-Zustände ein Fehler auftritt\nGrenzfall in der MULTI/EXEC-Logik des Redis-Simulators behoben\nVerhindern Sie einige von Sentry gemeldete Absturzfälle",
+        "ru": "Добавить отсутствующую зависимость axios\nРегистрировать ошибку только в том случае, если ошибка возникает при активации/деактивации адаптера через состояния .alive.\nИсправить пограничный случай в логике MULTI/EXEC симулятора Redis.\nПредотвращение некоторых случаев сбоев, о которых сообщает Sentry.",
+        "pt": "Adicionar dependência de axios ausente\nApenas registre um erro se ocorrer um erro ao ativar/desativar o adaptador por meio de estados .alive\nCorrigir caso de borda na lógica MULTI/EXEC do simulador redis\nPrevenir alguns casos de travamento relatados pelo Sentinela",
+        "nl": "Ontbrekende axios-afhankelijkheid toevoegen\nLog alleen een fout in als er een fout optreedt bij het activeren/deactiveren van de adapter via .alive states\nRandgeval repareren in redis-simulator MULTI/EXEC-logica\nVoorkom enkele door Sentry gemelde crashgevallen",
+        "fr": "Ajouter la dépendance axios manquante\nEnregistrez une erreur uniquement si une erreur se produit lors de l'activation/de la désactivation de l'adaptateur via les états .alive\nCorrection du cas limite dans la logique MULTI/EXEC du simulateur redis\nEmpêcher certains cas de plantage signalés par Sentry",
+        "it": "Aggiungi la dipendenza assio mancante\nRegistra un errore solo se si verifica un errore durante l'attivazione/disattivazione dell'adattatore tramite stati .alive\nRisolto il problema con il caso limite nella logica MULTI/EXEC del simulatore Redis\nPrevenire alcuni casi di crash segnalati da Sentry",
+        "es": "Agregar la dependencia de axios faltante\nSolo registre un error si ocurre un error al activar/desactivar el adaptador a través de estados .alive\nArreglar el caso de borde en la lógica MULTI / EXEC del simulador redis\nEvitar algunos casos de bloqueo informados por Sentry",
+        "pl": "Dodaj brakującą zależność axios\nZapisuj błąd tylko wtedy, gdy wystąpi błąd podczas aktywacji/dezaktywacji adaptera przez stany .alive\nNapraw obudowę krawędziową w symulatorze redis, logika MULTI/EXEC\nZapobiegaj niektórym przypadkom awarii zgłaszanych przez Sentry",
+        "zh-cn": "添加缺少的 axios 依赖项\n只有在通过 .alive 状态激活/停用适配器时发生错误时才记录错误\n修复 redis 模拟器 MULTI/EXEC 逻辑中的边缘情况\n防止 Sentry 报告的一些崩溃案例"
+      },
+      "4.0.19": {
         "en": "see CHANGELOG.md",
         "de": "siehe CHANGELOG.md",
         "ru": "см. CHANGELOG.md",

package/lib/setup/setupBackup.js

@@ -14,7 +14,6 @@ const pathLib = require('path');
 const hostname = tools.getHostName();
 const Upload = require('./setupUpload');
 const { EXIT_CODES } = require('@iobroker/js-controller-common');
-const path = require('path');
 const cpPromise = require('promisify-child-process');
 
 // We cannot use relative paths for the backup locations, as they used by both
@@ -125,14 +124,18 @@ class BackupRestore {
     copyFileSync(source, target) {
         let targetFile = target;
 
-        // if target is a directory a new file with the same name will be created
-        if (fs.existsSync(target)) {
-            if (fs.statSync(target).isDirectory()) {
-                targetFile = pathLib.join(target, pathLib.basename(source));
+        try {
+            // if target is a directory a new file with the same name will be created
+            if (fs.existsSync(target)) {
+                if (fs.statSync(target).isDirectory()) {
+                    targetFile = pathLib.join(target, pathLib.basename(source));
+                }
             }
-        }
 
-        fs.writeFileSync(targetFile, fs.readFileSync(source));
+            fs.writeFileSync(targetFile, fs.readFileSync(source));
+        } catch (e) {
+            console.error(`Could not copy ${targetFile} to ${source}: ${e.message}`);
+        }
     }
 
     copyFolderRecursiveSync(source, target) {
@@ -149,10 +152,13 @@ class BackupRestore {
         }
 
         // copy
-        if (fs.statSync(source).isDirectory()) {
+        if (fs.existsSync(source) && fs.statSync(source).isDirectory()) {
             files = fs.readdirSync(source);
             files.forEach(file => {
                 const curSource = pathLib.join(source, file);
+                if (!fs.existsSync(curSource)) {
+                    return;
+                }
                 if (fs.statSync(curSource).isDirectory()) {
                     this.copyFolderRecursiveSync(curSource, targetFolder);
                 } else {
@@ -600,7 +606,11 @@ class BackupRestore {
         }
         const files = fs.readdirSync(root + path);
         for (const file of files) {
-            const stat = fs.statSync(`${root + path}/${file}`);
+            const fName = pathLib.join(root, path, file);
+            if (!fs.existsSync(fName)) {
+                continue;
+            }
+            const stat = fs.statSync(fName);
             if (stat.isDirectory()) {
                 try {
                     await this._uploadUserFiles(root, `${path}/${file}`);
@@ -634,6 +644,9 @@ class BackupRestore {
                 return;
             }
             const path = pathLib.join(backupDir, dir);
+            if (!fs.existsSync(path)) {
+                return;
+            }
             const stat = fs.statSync(path);
             if (stat.isDirectory()) {
                 this.copyFolderRecursiveSync(path, this.configDir);
@@ -720,7 +733,7 @@ class BackupRestore {
             console.log('Forced restore - executing setup ...');
             try {
                 await cpPromise.exec(
-                    `"${process.execPath}" "${path.join(controllerDir, `${tools.appName.toLowerCase()}.js`)}" setup`
+                    `"${process.execPath}" "${pathLib.join(controllerDir, `${tools.appName.toLowerCase()}.js`)}" setup`
                 );
             } catch (e) {
                 console.error(
@@ -743,7 +756,7 @@ class BackupRestore {
      * @private
      */
     async _removeAllAdapters(controllerDir) {
-        const nodeModulePath = path.join(controllerDir, '..');
+        const nodeModulePath = pathLib.join(controllerDir, '..');
         const nodeModuleDirs = fs.readdirSync(nodeModulePath, { withFileTypes: true });
         // we need to uninstall current adapters to get exact the same system as before backup
         for (const dir of nodeModuleDirs) {
@@ -753,7 +766,7 @@ class BackupRestore {
                 dir.name !== `${tools.appName.toLowerCase()}.js-controller`
             ) {
                 try {
-                    const packJson = fs.readJsonSync(path.join(nodeModulePath, dir.name, 'package.json'));
+                    const packJson = fs.readJsonSync(pathLib.join(nodeModulePath, dir.name, 'package.json'));
                     console.log(`Removing current installation of ${packJson.name}`);
                     await tools.uninstallNodeModule(packJson.name);
                 } catch {
@@ -775,7 +788,7 @@ class BackupRestore {
      */
     _ensureCompatibility(controllerDir, backupHostname, backupObjects, force) {
         try {
-            const ioPackJson = fs.readJsonSync(path.join(controllerDir, 'io-package.json'));
+            const ioPackJson = fs.readJsonSync(pathLib.join(controllerDir, 'io-package.json'));
             const hostObj = backupObjects.find(obj => obj.id === `system.host.${backupHostname}`);
             if (hostObj.value.common.installedVersion !== ioPackJson.common.version) {
                 if (!force) {

package/lib/setup/setupUpload.js

@@ -90,10 +90,10 @@ function Upload(options) {
 
                 if (adapterConf.common.restartAdapters.length && adapterConf.common.restartAdapters[0]) {
                     const instances = await tools.getAllInstances(adapterConf.common.restartAdapters, objects);
-                    if (instances && !instances.length) {
-                        for (let r = 0; r < instances.length; r++) {
+                    if (instances && instances.length) {
+                        for (const instance of instances) {
                             try {
-                                const obj = await objects.getObjectAsync(instances[r]);
+                                const obj = await objects.getObjectAsync(instance);
                                 // if instance is enabled
                                 if (obj && obj.common && obj.common.enabled) {
                                     obj.common.enabled = false; // disable instance
@@ -101,7 +101,7 @@ function Upload(options) {
                                     obj.from = `system.host.${tools.getHostName()}.cli`;
                                     obj.ts = Date.now();
 
-                                    await objects.setObjectAsync(obj._id);
+                                    await objects.setObjectAsync(obj._id, obj);
 
                                     obj.common.enabled = true; // enable instance
                                     obj.ts = Date.now();
@@ -110,7 +110,7 @@ function Upload(options) {
                                     console.log(`Adapter "${obj._id}" restarted.`);
                                 }
                             } catch (err) {
-                                console.error(`Cannot restart adapter "${instances[r]}": ${err.message}`);
+                                console.error(`Cannot restart adapter "${instance}": ${err.message}`);
                             }
                         }
                     }

package/main.js

@@ -408,8 +408,10 @@ function createStates(onConnect) {
                                 );
                                 obj.from = hostObjectPrefix;
                                 obj.ts = Date.now();
-                                objects.setObject(obj._id, obj, err =>
-                                    logger.error(`${hostLogPrefix} Cannot set object: ${err}`)
+                                objects.setObject(
+                                    obj._id,
+                                    obj,
+                                    err => err && logger.error(`${hostLogPrefix} Cannot set object: ${err}`)
                                 );
                             }
                         }
@@ -1464,6 +1466,8 @@ function setIPs(ipList) {
                 !err &&
                 oldObj &&
                 oldObj.common &&
+                oldObj.native &&
+                oldObj.native.hardware &&
                 (!isDeepStrictEqual(oldObj.native.hardware.networkInterfaces, networkInterfaces) ||
                     !isDeepStrictEqual(oldObj.common.address, _ipList))
             ) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "iobroker.js-controller",
-  "version": "4.0.18",
+  "version": "4.0.21",
   "engines": {
     "node": ">=12.0.0"
   },
@@ -17,16 +17,16 @@
   },
   "dependencies": {
     "@alcalzone/esbuild-register": "^2.5.1-1",
-    "@iobroker/db-objects-file": "4.0.18",
-    "@iobroker/db-objects-jsonl": "4.0.18",
-    "@iobroker/db-objects-redis": "4.0.18",
-    "@iobroker/db-states-file": "4.0.18",
-    "@iobroker/db-states-jsonl": "4.0.18",
-    "@iobroker/db-states-redis": "4.0.18",
-    "@iobroker/js-controller-adapter": "4.0.18",
-    "@iobroker/js-controller-cli": "4.0.18",
-    "@iobroker/js-controller-common": "4.0.18",
-    "@iobroker/js-controller-common-db": "4.0.18",
+    "@iobroker/db-objects-file": "4.0.21",
+    "@iobroker/db-objects-jsonl": "4.0.21",
+    "@iobroker/db-objects-redis": "4.0.21",
+    "@iobroker/db-states-file": "4.0.21",
+    "@iobroker/db-states-jsonl": "4.0.21",
+    "@iobroker/db-states-redis": "4.0.21",
+    "@iobroker/js-controller-adapter": "4.0.21",
+    "@iobroker/js-controller-cli": "4.0.21",
+    "@iobroker/js-controller-common": "4.0.21",
+    "@iobroker/js-controller-common-db": "4.0.21",
     "@iobroker/plugin-base": "~1.2.1",
     "@iobroker/plugin-sentry": "~1.2.0",
     "axios": "^0.24.0",
@@ -112,5 +112,5 @@
     "iobroker.js",
     "main.js"
   ],
-  "gitHead": "ff1f5de7065853138ffc0ece3dc941c7a59fec6c"
+  "gitHead": "4749d256d312623ae7621e7ae47d44580b3aa8c1"
 }