Only output ipv4 addresses + localhost and :: when starting dev server

Author: jspspike

.changeset/small-gifts-smash.md

@@ -0,0 +1,5 @@
+---
+"miniflare": patch
+---
+
+Only output ipv4 addresses when starting

packages/miniflare/src/http/helpers.ts

@@ -9,8 +9,8 @@ export function getAccessibleHosts(ipv4Only = false): string[] {
 			// @ts-expect-error the `family` property is numeric as of Node.js 18.0.0
 			if (family === "IPv4" || family === 4) {
 				hosts.push(address);
-			} else if (!ipv4Only && (family === "IPv6" || family === 6)) {
-				hosts.push(`[${address}]`);
+			} else if (!ipv4Only) {
+				hosts.push(address);
 			}
 		});
 	});

packages/miniflare/src/index.ts

@@ -1181,13 +1181,14 @@ export class Miniflare {
 			);
 
 			if (initial) {
-				let hosts: string[];
-				if (host === "::" || host === "*") {
-					hosts = getAccessibleHosts(false);
-				} else if (host === "0.0.0.0") {
-					hosts = getAccessibleHosts(true);
-				} else {
-					hosts = [];
+				const hosts: string[] = [];
+				if (host === "::" || host === "*" || host === "0.0.0.0") {
+					hosts.push(...getAccessibleHosts(true));
+
+					if (host !== "0.0.0.0") {
+						hosts.push("localhost");
+						hosts.push("[::1]");
+					}
 				}
 
 				for (const h of hosts) {

packages/wrangler/src/dev/proxy.ts

@@ -2,8 +2,8 @@ import { createServer as createHttpServer } from "node:http";
 import { connect } from "node:http2";
 import { createServer as createHttpsServer } from "node:https";
 import https from "node:https";
-import { networkInterfaces } from "node:os";
 import { createHttpTerminator } from "http-terminator";
+import { getAccessibleHosts } from "miniflare";
 import { useEffect, useRef, useState } from "react";
 import serveStatic from "serve-static";
 import { getHttpsOptions } from "../https-options";
@@ -166,7 +166,15 @@ export async function startPreviewServer({
 			const usedPort =
 				address && typeof address === "object" ? address.port : port;
 			logger.log(`⬣ Listening at ${localProtocol}://${ip}:${usedPort}`);
-			const accessibleHosts = getAccessibleHosts(ip);
+			const accessibleHosts = [];
+			if (ip === "::" || ip === "*" || ip === "0.0.0.0") {
+				accessibleHosts.push(...getAccessibleHosts(true));
+
+				if (ip !== "0.0.0.0") {
+					accessibleHosts.push("localhost");
+					accessibleHosts.push("[::1]");
+				}
+			}
 			for (const accessibleHost of accessibleHosts) {
 				logger.log(`- ${localProtocol}://${accessibleHost}:${usedPort}`);
 			}
@@ -299,7 +307,15 @@ export function usePreviewServer({
 					const usedPort =
 						address && typeof address === "object" ? address.port : port;
 					logger.log(`⬣ Listening at ${localProtocol}://${ip}:${usedPort}`);
-					const accessibleHosts = getAccessibleHosts(ip);
+					const accessibleHosts = [];
+					if (ip === "::" || ip === "*" || ip === "0.0.0.0") {
+						accessibleHosts.push(...getAccessibleHosts(true));
+
+						if (ip !== "0.0.0.0") {
+							accessibleHosts.push("localhost");
+							accessibleHosts.push("[::1]");
+						}
+					}
 					for (const accessibleHost of accessibleHosts) {
 						logger.log(`- ${localProtocol}://${accessibleHost}:${usedPort}`);
 					}
@@ -681,24 +697,3 @@ export async function waitForPortToBeAvailable(
 		}
 	});
 }
-
-function getAccessibleHosts(ip: string): string[] {
-	if (ip !== "0.0.0.0" && ip !== "::" && ip !== "*") {
-		return [ip];
-	}
-
-	const hosts: string[] = [];
-	Object.values(networkInterfaces()).forEach((net) => {
-		net?.forEach(({ family, address }) => {
-			if (ip === "0.0.0.0") {
-				// @ts-expect-error the `family` property is numeric as of Node.js 18.0.0
-				if (family === "IPv4" || family === 4) {
-					hosts.push(address);
-				}
-			} else {
-				hosts.push(address);
-			}
-		});
-	});
-	return hosts;
-}

packages/wrangler/src/https-options.ts

@@ -1,7 +1,7 @@
 import * as fs from "node:fs";
-import os from "node:os";
 import * as path from "node:path";
 import { promisify } from "node:util";
+import { getAccessibleHosts } from "miniflare";
 import { getGlobalWranglerConfigPath } from "./global-wrangler-config-path";
 import { logger } from "./logger";
 import type { Attributes, Options } from "selfsigned";
@@ -103,7 +103,7 @@ async function generateCertificate() {
 				name: "subjectAltName",
 				altNames: [
 					{ type: 2, value: "localhost" },
-					...getAccessibleHosts().map((ip) => ({ type: 7, ip })),
+					...getAccessibleHosts(false).map((ip) => ({ type: 7, ip })),
 				],
 			},
 		],
@@ -112,16 +112,3 @@ async function generateCertificate() {
 	const { private: key, cert } = await generate(certAttrs, certOptions);
 	return { key, cert };
 }
-
-/**
- * Ask the OS for the addresses of locally accessible hosts.
- */
-function getAccessibleHosts(ipv4 = false): string[] {
-	const hosts: string[] = [];
-	Object.values(os.networkInterfaces()).forEach((net) =>
-		net?.forEach(({ family, address }) => {
-			if (!ipv4 || family === "IPv4") hosts.push(address);
-		})
-	);
-	return hosts;
-}