homebridge-xbox-tv 3.3.1 → 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/package.json

@@ -1,7 +1,7 @@
 {
   "displayName": "Xbox TV",
   "name": "homebridge-xbox-tv",
-  "version": "3.3.1",
+  "version": "3.3.2",
   "description": "Homebridge plugin to control Xbox game consoles.",
   "license": "MIT",
   "author": "grzegorz914",
@@ -31,7 +31,7 @@
     "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": {

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}`)
         }