Revert change to onConnectHostname

Author: hansott

library/agent/Agent.ts

@@ -565,8 +565,8 @@ export class Agent {
     }
   }
 
-  onConnectHostname(url: URL, port: number) {
-    this.hostnames.add(url.hostname, port);
+  onConnectHostname(hostname: string, port: number) {
+    this.hostnames.add(hostname, port);
   }
 
   onConnectHTTP(url: URL, port: number, method: string) {

library/sinks/Fetch.ts

@@ -23,7 +23,7 @@ export class Fetch implements Wrapper {
     // Let the agent know that we are connecting to this hostname
     // This is to build a list of all hostnames that the application is connecting to
     if (typeof port === "number" && port > 0) {
-      agent.onConnectHostname(url, port);
+      agent.onConnectHostname(url.hostname, port);
       agent.onConnectHTTP(url, port, method);
     }
     const context = getContext();
@@ -57,7 +57,12 @@ export class Fetch implements Wrapper {
       if (typeof args[0] === "string" && args[0].length > 0) {
         const url = tryParseURL(args[0]);
         if (url) {
-          const attack = this.inspectHostname(agent, url, getPortFromURL(url), method);
+          const attack = this.inspectHostname(
+            agent,
+            url,
+            getPortFromURL(url),
+            method
+          );
           if (attack) {
             return attack;
           }
@@ -71,7 +76,12 @@ export class Fetch implements Wrapper {
       if (Array.isArray(args[0])) {
         const url = tryParseURL(args[0].toString());
         if (url) {
-          const attack = this.inspectHostname(agent, url, getPortFromURL(url), method);
+          const attack = this.inspectHostname(
+            agent,
+            url,
+            getPortFromURL(url),
+            method
+          );
           if (attack) {
             return attack;
           }
@@ -95,7 +105,12 @@ export class Fetch implements Wrapper {
       if (args[0] instanceof Request) {
         const url = tryParseURL(args[0].url);
         if (url) {
-          const attack = this.inspectHostname(agent, url, getPortFromURL(url), method);
+          const attack = this.inspectHostname(
+            agent,
+            url,
+            getPortFromURL(url),
+            method
+          );
           if (attack) {
             return attack;
           }

library/sinks/HTTPRequest.ts

@@ -25,7 +25,7 @@ export class HTTPRequest implements Wrapper {
     // Let the agent know that we are connecting to this hostname
     // This is to build a list of all hostnames that the application is connecting to
     if (typeof port === "number" && port > 0) {
-      agent.onConnectHostname(url, port);
+      agent.onConnectHostname(url.hostname, port);
       agent.onConnectHTTP(url, port, method);
     }
     const context = getContext();

library/sinks/Undici.ts

@@ -26,14 +26,14 @@ const methods = [
 export class Undici implements Wrapper {
   private inspectHostname(
     agent: Agent,
-    url: URL,
+    hostname: string,
     port: number | undefined,
     method: string
   ): InterceptorResult {
     // Let the agent know that we are connecting to this hostname
     // This is to build a list of all hostnames that the application is connecting to
     if (typeof port === "number" && port > 0) {
-      agent.onConnectHostname(url, port);
+      agent.onConnectHostname(hostname, port);
     }
     const context = getContext();
 
@@ -42,7 +42,7 @@ export class Undici implements Wrapper {
     }
 
     return checkContextForSSRF({
-      hostname: url.hostname,
+      hostname: hostname,
       operation: `undici.${method}`,
       context,
       port,