Fix homebridge#945 - Node.js v18 os.networkInterfaces()[].family numb…

…er/string comparison (homebridge#947) Co-authored-by: Andi <mail@anderl-bauer.de>
koush · Jun 28, 2022 · fc381e0 · fc381e0
1 parent 4b1cc68
commit fc381e0
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/src/lib/util/eventedhttp.ts b/src/lib/util/eventedhttp.ts
@@ -686,7 +686,8 @@ export class HAPConnection extends EventEmitter {
 
     if (ipVersion === "ipv4") {
       for (const info of infos) {
-        if (info.family === "IPv4") {
+        // @ts-expect-error Nodejs 18+ uses the number 4 the string "IPv4"
+        if (info.family === "IPv4" || info.family === 4) {
           return info.address;
         }
       }
@@ -696,7 +697,8 @@ export class HAPConnection extends EventEmitter {
       let localUniqueAddress: string | undefined = undefined;
 
       for (const info of infos) {
-        if (info.family === "IPv6") {
+        // @ts-expect-error Nodejs 18+ uses the number 6 instead of the string "IPv6"
+        if (info.family === "IPv6" || info.family === 6) {
           if (!info.scopeid) {
             return info.address;
           } else if (!localUniqueAddress) {

diff --git a/src/lib/util/net-utils.ts b/src/lib/util/net-utils.ts
@@ -16,11 +16,13 @@ export function findLoopbackAddress(): string {
       }
 
       internal = true;
-      if (info.family === "IPv4") {
+      // @ts-expect-error Nodejs 18+ uses the number 4 the string "IPv4"
+      if (info.family === "IPv4" || info.family === 4) {
         if (!ipv4) {
           ipv4 = info.address;
         }
-      } else if (info.family === "IPv6") {
+      // @ts-expect-error Nodejs 18+ uses the number 6 the string "IPv6"
+      } else if (info.family === "IPv6" || info.family === 6) {
         if (info.scopeid) {
           if (!ipv6LinkLocal) {
             ipv6LinkLocal = info.address + "%" + name; // ipv6 link local addresses are only valid with a scope