Working on exception caused by wrong config

Author: En3rGy

package.json

@@ -1,7 +1,7 @@
 {
   "name": "homebridge-hsd",
   "displayName": "Homebridge Gira Homeserver URL-Endpoint",
-  "version": "1.1.5",
+  "version": "1.1.6",
   "description": "Plugin to access KNX bus via Gira Homeserver",
   "license": "MIT",
   "keywords": [

src/hsdAccessory.ts

@@ -63,9 +63,7 @@ class HsdAccessory {
   public setupServices (accessory: HsdPlatformAccessory): void {
     for (const service of accessory.context.services) {
       this.logger.info('Set up service', accessory.context.accessoryName, service.serviceName);
-
       switch (service.serviceType) {
-
         case 'GarageDoorOpener':
           this.services.push(new GarageDoorOpener(this.api, this.hsd, accessory, service));
           break;

src/hsdPlatform.ts

@@ -10,15 +10,22 @@ import { HsdPlatformConfig } from './config';
 export class HsdPlatform implements DynamicPlatformPlugin {
   private cachedAccessories: Map<string, HsdPlatformAccessory> = new Map();
   private hsdAccessories: Map<string, HsdAccessory> = new Map();
-  private config: HsdPlatformConfig;
+  private config: HsdPlatformConfig = {
+    hsIp: '',
+    hsUserName: '',
+    hsUserPw: '',
+    hsPort: 0,
+    accessories: [], // Assuming you have an array of HsdAccessoryConfig objects
+    platform: '',
+  };
 
   private async connect (): Promise<HomeServerConnector> {
     const link = HomeServerConnector.getInstance(this.api, this.logger, this.hsdAccessories);
     link.connect(this.config.hsIp, this.config.hsPort, this.config.hsUserName, this.config.hsUserPw);
     this.logger.info(`hsdPlatform.ts | HsdPlatform | HSD IP gateway ${this.config.hsIp} connection established.`);
 
     this.api.on(APIEvent.SHUTDOWN, async () => {
-      await link.disconnect();
+      link.disconnect();
       this.logger.warn(`hsdPlatform.ts | HsdPlatform | hsd IP gateway ${this.config.hsdIp} connection closed.`);
     });
 
@@ -27,10 +34,11 @@ export class HsdPlatform implements DynamicPlatformPlugin {
 
   public constructor (private logger: Logging, config: PlatformConfig, private api: API) {
     this.logger.debug('hsdPlatform.ts | HsdPlatform | Constructor');
-    if (!isHsdPlatformConfig(config)) {
-      throw new Error('hsdPlatform.ts | HsdPlatform | Invalid configuration');
-    } else {
+    if (isHsdPlatformConfig(config)) {
+
       this.config = config;
+    } else {
+      this.logger.error('hsdPlatform.ts | HsdPlatform | Invalid configuration');
     }
 
     api.on(APIEvent.DID_FINISH_LAUNCHING, async () => {
@@ -44,9 +52,13 @@ export class HsdPlatform implements DynamicPlatformPlugin {
 
   private configureAccessories (hsd: HomeServerConnector): void {
     for (const config of this.config.accessories) {
-      const hsdAccessory = new HsdAccessory(config, this.logger, hsd, this.api);
-      this.hsdAccessories.set(hsdAccessory.uuid, hsdAccessory);
-      this.logger.info('hsdPlatform.ts | HsdPlatform | Loaded hsd accessory', hsdAccessory.displayName);
+      try {
+        const hsdAccessory = new HsdAccessory(config, this.logger, hsd, this.api);
+        this.hsdAccessories.set(hsdAccessory.uuid, hsdAccessory);
+        this.logger.info('hsdPlatform.ts | HsdPlatform | Loaded hsd accessory', hsdAccessory.displayName);
+      } catch (error) {
+        this.logger.error('hsdPlatform.ts | HsdPlatform | Error loading accessory', error);
+      }
     }
 
     for (const accessory of this.cachedAccessories.values()) {
@@ -61,7 +73,6 @@ export class HsdPlatform implements DynamicPlatformPlugin {
 
       try {
         hsdAccessory.setupServices(accessory);
-
       } catch (e) {
         this.api.unregisterPlatformAccessories(PLUGIN_NAME, PLATFORM_NAME, [accessory]);
         this.logger.debug('hsdPlatform.ts | HsdPlatform | Unregistered hsd accessory', accessory.displayName);

src/hsdPlatformAccessory.ts

@@ -4,5 +4,11 @@ import { HsdAccessoryConfig, HsdPlatformConfig } from './config';
 export type HsdPlatformAccessory = PlatformAccessory<HsdAccessoryConfig>;
 
 export const isHsdPlatformConfig = (config: PlatformConfig): config is HsdPlatformConfig => {
-  return 'hsIp' in config;
+  let isConfig = true;
+  isConfig = isConfig && 'hsIp' in config;
+  isConfig = isConfig && 'hsUserName' in config;
+  isConfig = isConfig && 'hsUserPw' in config;
+  isConfig = isConfig && 'hsPort' in config;
+
+  return isConfig;
 };