add unit test, changeset

.changeset/gold-bugs-sneeze.md

@@ -0,0 +1,5 @@
+---
+"@smithy/node-http-handler": patch
+---
+
+remove brackets from hostname

packages/node-http-handler/src/node-http-handler.spec.ts

@@ -153,6 +153,24 @@ describe("NodeHttpHandler", () => {
         expect(hRequestSpy.mock.calls[0][0]?.port).toEqual(1234);
         expect(hRequestSpy.mock.calls[0][0]?.path).toEqual("/some/path?some=query#fragment");
       });
+
+      it("removes brackets from hostname", async () => {
+        const nodeHttpHandler = new NodeHttpHandler({});
+        const httpRequest = {
+          protocol: "http:",
+          username: "username",
+          password: "password",
+          hostname: "[host]",
+          port: 1234,
+          path: "/some/path",
+          query: {
+            some: "query",
+          },
+          fragment: "fragment",
+        };
+        await nodeHttpHandler.handle(httpRequest as any);
+        expect(hRequestSpy.mock.calls[0][0]?.host).toEqual("host");
+      });
     });
   });
 

packages/node-http-handler/src/node-http-handler.ts

@@ -4,6 +4,7 @@ import type { Logger, NodeHttpHandlerOptions } from "@smithy/types";
 import { HttpHandlerOptions, Provider } from "@smithy/types";
 import { Agent as hAgent, request as hRequest } from "http";
 import { Agent as hsAgent, request as hsRequest, RequestOptions } from "https";
+import { urlToHttpOptions } from "url";
 
 import { NODEJS_TIMEOUT_ERROR_CODES } from "./constants";
 import { getTransformedHeaders } from "./get-transformed-headers";
@@ -210,8 +211,8 @@ or increase socketAcquisitionWarningTimeout=(millis) in the NodeHttpHandler conf
         path += `#${request.fragment}`;
       }
 
-      let hostname: string;
-      if (request.hostname.startsWith("[") && request.hostname.endsWith("]")) {
+      let hostname = request.hostname ?? "";
+      if (hostname[0] === "[" && hostname.endsWith("]")) {
         hostname = request.hostname.slice(1, -1);
       } else {
         hostname = request.hostname;