fix #86

CHANGELOG.md

@@ -2,6 +2,12 @@
 
 All notable changes to this project will be documented in this file. This project uses [Semantic Versioning](https://semver.org/).
 
+## 3.5.5 (2018-04-21)
+
+### Bug Fixes
+
+* Ensure forked process is killed if Homebridge is not running in a different way (@Damien via Slack & [#86](https://github.com/oznu/homebridge-config-ui-x/issues/86))
+
 ## 3.5.4 (2018-04-19)
 
 ### Notable Changes

package.json

@@ -1,6 +1,6 @@
 {
   "name": "homebridge-config-ui-x",
-  "version": "3.5.4",
+  "version": "3.5.5",
   "description": "Configuration UI plugin for Homebridge",
   "license": "MIT",
   "keywords": [

src/bin/fork.ts

@@ -4,19 +4,19 @@ import 'source-map-support/register';
 
 import { UiServer } from '../server';
 
-let healthcheck;
+setInterval(() => {
+  if (!process.connected) {
+    process.exit(1);
+  }
+}, 10000);
 
 process.on('message', (message) => {
   if (typeof message === 'object') {
     return new UiServer(message);
-  } else if (message === 'ping') {
-    clearTimeout(healthcheck);
-    healthcheck = setTimeout(process.exit, 30000);
   }
 });
 
 process.on('disconnect', () => {
-  console.warn('Parent process terminated, shutting down homebridge-config-ui-x...');
   process.exit();
 });
 

src/index.ts

@@ -47,7 +47,6 @@ class HomebridgeConfigUi {
 
     ui.on('message', (message) => {
       if (message === 'ready') {
-        ui.send('ping');
         ui.send(setup);
       }
     });
@@ -56,9 +55,7 @@ class HomebridgeConfigUi {
       process.exit(1);
     });
 
-    setInterval(() => {
-      ui.send('ping');
-    }, 15000);
+    ui.on('error', (err) => {});
   }
 
   /**

src/server.ts

@@ -44,7 +44,7 @@ export class UiServer {
   onServerListening() {
     const addr = this.server.address();
     const bind = typeof addr === 'string' ? 'pipe ' + addr : 'port ' + addr.port;
-    const msg = 'Console is listening on ' + bind + '.';
+    const msg = `Console v${hb.ui.version} is listening on ${bind}.`;
     hb.log(msg);
   }