Mitigate event emitter "memory leak" warnings when a significant numb…

…er of HSV events occur simultaneously. (#1029)
homebridge · Apr 28, 2024 · 19f80ab · 19f80ab
1 parent 5cab65c
commit 19f80ab
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/src/lib/datastream/DataStreamServer.ts b/src/lib/datastream/DataStreamServer.ts
@@ -804,6 +804,7 @@ export class DataStreamConnection extends EventEmitter {
           this.state = ConnectionState.EXPECTING_HELLO;
 
           // below listener is removed in .close()
+          this.connection.setMaxListeners(this.connection.getMaxListeners() + 1);
           this.connection.on(HAPConnectionEvent.CLOSED, this.hapConnectionClosedListener); // register close listener
 
           debug("[%s] Registering CLOSED handler to HAP connection. Connection currently has %d close handlers!",
@@ -1139,6 +1140,7 @@ export class DataStreamConnection extends EventEmitter {
     this.emit(DataStreamConnectionEvent.CLOSED);
 
     this.connection?.removeListener(HAPConnectionEvent.CLOSED, this.hapConnectionClosedListener);
+    this.connection?.setMaxListeners(this.connection.getMaxListeners() - 1);
     this.removeAllListeners();
   }