Mitigate event emitter "memory leak" warnings when a significant number of HomeKit camera streaming events occur simultaneously. (#1037)

bwp91 · bwp91 · commit 279a86c18b97 · 2025-06-01T14:30:29.000+01:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,12 @@
 
 All notable changes to `hap-nodejs` will be documented in this file. This project tries to adhere to [Semantic Versioning](http://semver.org/).
 
+## v0.13.0 (Unreleased)
+
+### Changed
+
+- Mitigate event emitter "memory leak" warnings when a significant number of HomeKit camera streaming events occur simultaneously. (#1037)
+
 ## v0.12.3 (2024-10-26)
 
 ### Changed
diff --git a/src/lib/camera/RTPStreamManagement.ts b/src/lib/camera/RTPStreamManagement.ts
@@ -784,6 +784,7 @@ export class RTPStreamManagement {
 
     if (this.activeConnection) {
       this.activeConnection.removeListener(HAPConnectionEvent.CLOSED, this.activeConnectionClosedListener);
+      this.activeConnection.setMaxListeners(this.activeConnection.getMaxListeners() - 1);
       this.activeConnection = undefined;
     }
 
@@ -1098,6 +1099,7 @@ export class RTPStreamManagement {
       "Found non-nil `activeConnection` when trying to setup streaming endpoints, even though streamStatus is reported to be AVAILABLE!");
 
     this.activeConnection = connection;
+    this.activeConnection.setMaxListeners(this.activeConnection.getMaxListeners() + 1);
     this.activeConnection.on(HAPConnectionEvent.CLOSED, this.activeConnectionClosedListener);
 
     // noinspection JSDeprecatedSymbols
