homebridge-xbox-tv 3.3.0 → 3.3.2

package/CHANGELOG.md

@@ -10,6 +10,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - After update to 2.x.x the plugin settings (xboxLiveId) need to be updated
 - After update to v3.0.0 RESTFull and MQTT config settings need to be updated
 
+## [3.3.2] - (04.02.2025)
+
+## Changes
+
+- update RESTFul
+- bump dependencies
+
 ## [3.3.0] - (19.01.2025)
 
 ## Changes

package/index.js

@@ -101,7 +101,7 @@ class XboxPlatform {
 							const emitLog = !enableDebugMode ? false : log.info(`Device: ${host} ${deviceName}, debug: ${debug}.`);
 						})
 						.on('warn', (warn) => {
-							const lemitLogog = disableLogWarn ? false : log.warn(`Device: ${host} ${deviceName}, ${warn}.`);
+							const emitLog = disableLogWarn ? false : log.warn(`Device: ${host} ${deviceName}, ${warn}.`);
 						})
 						.on('error', (error) => {
 							const emitLog = disableLogError ? false : log.error(`Device: ${host} ${deviceName}, ${error}.`);
@@ -112,7 +112,7 @@ class XboxPlatform {
 					impulseGenerator.on('start', async () => {
 						try {
 							const startDone = await xboxDevice.start();
-							const stopImpulseGenerator = startDone ? impulseGenerator.stop() : false;
+							const stopImpulseGenerator = startDone ? await impulseGenerator.stop() : false;
 						} catch (error) {
 							const emitLog = disableLogError ? false : log.error(`Device: ${host} ${deviceName}, ${error}, trying again.`);
 						};
@@ -121,7 +121,7 @@ class XboxPlatform {
 					});
 
 					//start impulse generator
-					impulseGenerator.start([{ name: 'start', sampling: 45000 }]);
+					await impulseGenerator.start([{ name: 'start', sampling: 45000 }]);
 				} catch (error) {
 					throw new Error(`Device: ${host} ${deviceName}, Did finish launching error: ${error}.`);
 				}

package/package.json

@@ -1,7 +1,7 @@
 {
   "displayName": "Xbox TV",
   "name": "homebridge-xbox-tv",
-  "version": "3.3.0",
+  "version": "3.3.2",
   "description": "Homebridge plugin to control Xbox game consoles.",
   "license": "MIT",
   "author": "grzegorz914",
@@ -31,11 +31,11 @@
     "LICENSE"
   ],
   "engines": {
-    "homebridge": "^1.6.0 || ^2.0.0-beta.0",
+    "homebridge": "^1.8.5 || ^2.0.0-beta.0",
     "node": "^18.20.4 || ^20.15.1 || ^22.7.0 || ^23.2.0"
   },
   "dependencies": {
-    "@homebridge/plugin-ui-utils": "^2.0.0",
+    "@homebridge/plugin-ui-utils": "^2.0.1",
     "async-mqtt": "^2.6.3",
     "elliptic": "^6.6.1",
     "hex-to-binary": "^1.0.1",

package/src/restful.js

@@ -24,6 +24,8 @@ class RestFul extends EventEmitter {
             const restFul = express();
             restFul.set('json spaces', 2);
             restFul.use(json());
+
+            // GET Routes
             restFul.get('/info', (req, res) => { res.json(this.restFulData.info) });
             restFul.get('/state', (req, res) => { res.json(this.restFulData.state) });
             restFul.get('/consoleslist', (req, res) => { res.json(this.restFulData.consoleslist) });
@@ -32,24 +34,30 @@ class RestFul extends EventEmitter {
             restFul.get('/storages', (req, res) => { res.json(this.restFulData.storages) });
             restFul.get('/status', (req, res) => { res.json(this.restFulData.status) });
 
-            //post data
+            // POST Route
             restFul.post('/', (req, res) => {
                 try {
                     const obj = req.body;
-                    const emitDebug = this.restFulDebug ? this.emit('debug', `RESTFul post data: ${JSON.stringify(obj, null, 2)}`) : false;
+                    if (!obj || typeof obj !== 'object' || Object.keys(obj).length === 0) {
+                        this.emit('warn', `RESTFul Invalid JSON payload`);
+                        return res.status(400).json({ error: 'RESTFul Invalid JSON payload' });
+                    }
                     const key = Object.keys(obj)[0];
-                    const value = Object.values(obj)[0];
+                    const value = obj[key];
                     this.emit('set', key, value);
-                    res.send('OK');
+
+                    const emitDebug = this.restFulDebug ? this.emit('debug', `RESTFul post data: ${JSON.stringify(obj, null, 2)}`) : false;
+                    res.json({ success: true, received: obj });
                 } catch (error) {
-                    this.emit('warn', `RESTFul Parse object error: ${error}`);
-                };
+                    this.emit('warn', `RESTFul Parse error: ${error}`);
+                    res.status(500).json({ error: 'RESTFul Internal Server Error' });
+                }
             });
 
+            // Start server
             restFul.listen(this.restFulPort, () => {
-                this.emit('connected', `RESTful started on port: ${this.restFulPort}`)
+                this.emit('connected', `RESTful started on port: ${this.restFulPort}`);
             });
-
         } catch (error) {
             this.emit('warn', `RESTful Connect error: ${error}`)
         }