refactor(request): align jsdocs with getRequestHost (#1316)

Author: productdevbook

docs/2.utils/1.request.md

@@ -139,7 +139,7 @@ Get the request hostname.
 
 If `xForwardedHost` is `true`, it will use the `x-forwarded-host` header if it exists.
 
-If no host header is found, it will default to "localhost".
+If no host header is found, it will return an empty string.
 
 **Example:**
 

src/utils/request.ts

@@ -319,7 +319,7 @@ export function assertMethod(
  *
  * If `xForwardedHost` is `true`, it will use the `x-forwarded-host` header if it exists.
  *
- * If no host header is found, it will default to "localhost".
+ * If no host header is found, it will return an empty string.
  *
  * @example
  * app.get("/", (event) => {

test/utils.test.ts

@@ -5,6 +5,7 @@ import {
   assertMethod,
   getQuery,
   getRequestURL,
+  getRequestHost,
   getRequestIP,
   getRequestFingerprint,
   handleCacheHeaders,
@@ -83,6 +84,31 @@ describeMatrix("utils", (t, { it, describe, expect }) => {
     });
   });
 
+  describe("getRequestHost", () => {
+    it("returns host header value", async () => {
+      t.app.get("/", (event) => getRequestHost(event));
+      const res = await t.fetch("/");
+      // In test environments, host header is set by the HTTP client
+      expect(await res.text()).toBeTruthy();
+    });
+
+    it("uses x-forwarded-host when enabled", async () => {
+      t.app.get("/", (event) => getRequestHost(event, { xForwardedHost: true }));
+      const res = await t.fetch("/", {
+        headers: { "x-forwarded-host": "proxy.example.com" },
+      });
+      expect(await res.text()).toBe("proxy.example.com");
+    });
+
+    it("uses first value from x-forwarded-host with multiple entries", async () => {
+      t.app.get("/", (event) => getRequestHost(event, { xForwardedHost: true }));
+      const res = await t.fetch("/", {
+        headers: { "x-forwarded-host": "first.com, second.com" },
+      });
+      expect(await res.text()).toBe("first.com");
+    });
+  });
+
   describe("getRequestURL", () => {
     const tests = [
       "http://localhost/foo?bar=baz",