Merge 4340181a31 into 4583f88626

src/media/host.ts

@@ -44,7 +44,7 @@ export async function ensureMediaHosted(
     if (!mediaServer) {
       mediaServer = await startMediaServer(port, TTL_MS, runtime);
       logInfo(
-        `🦞 Started temporary media host on http://localhost:${port}/media/:id (TTL ${TTL_MS / 1000}s)`,
+        `🦞 Started temporary media host on http://127.0.0.1:${port}/media/:id (TTL ${TTL_MS / 1000}s)`,
         runtime,
       );
       mediaServer.unref?.();

src/media/server.test.ts

@@ -54,6 +54,13 @@ describe("media server", () => {
     await new Promise((r) => server.close(r));
   });
 
+  it("binds to loopback by default", async () => {
+    const server = await startMediaServer(0, 5_000);
+    const addr = server.address() as AddressInfo;
+    expect(["127.0.0.1", "::1"]).toContain(addr.address);
+    await new Promise((r) => server.close(r));
+  });
+
   it("expires old media", async () => {
     const file = path.join(MEDIA_DIR, "old");
     await fs.writeFile(file, "stale");

src/media/server.ts

@@ -84,11 +84,12 @@ export async function startMediaServer(
   port: number,
   ttlMs = DEFAULT_TTL_MS,
   runtime: RuntimeEnv = defaultRuntime,
+  host = "127.0.0.1",
 ): Promise<Server> {
   const app = express();
   attachMediaRoutes(app, ttlMs, runtime);
   return await new Promise((resolve, reject) => {
-    const server = app.listen(port);
+    const server = app.listen(port, host);
     server.once("listening", () => resolve(server));
     server.once("error", (err) => {
       runtime.error(danger(`Media server failed: ${String(err)}`));