move shouldPassThrough to util file

Author: lipsumar

src/server/http-combo-server.ts

@@ -19,6 +19,7 @@ import { URLPattern } from "urlpattern-polyfill";
 import { TlsHandshakeFailure } from '../types';
 import { getCA } from '../util/tls';
 import { delay } from '../util/util';
+import { shouldPassThrough } from '../util/server-utils';
 import {
     getParentSocket,
     buildSocketTimingInfo,
@@ -421,22 +422,3 @@ function analyzeAndMaybePassThroughTls(
         tlsConnectionListener.call(server, socket);
     });
 }
-
-export function shouldPassThrough(
-    hostname: string | undefined,
-    // Only one of these two should have values (validated above):
-    passThroughPatterns: URLPattern[],
-    interceptOnlyPatterns: URLPattern[] | undefined
-  ): boolean {
-    if (!hostname) return false;
-  
-    if (interceptOnlyPatterns) {
-      return !interceptOnlyPatterns.some((pattern) =>
-        pattern.test(`https://${hostname}`)
-      );
-    }
-  
-    return passThroughPatterns.some((pattern) =>
-        pattern.test(`https://${hostname}`)
-    );
-}

src/util/server-utils.ts

@@ -0,0 +1,18 @@
+export function shouldPassThrough(
+  hostname: string | undefined,
+  // Only one of these two should have values (validated above):
+  passThroughPatterns: URLPattern[],
+  interceptOnlyPatterns: URLPattern[] | undefined
+): boolean {
+  if (!hostname) return false;
+
+  if (interceptOnlyPatterns) {
+    return !interceptOnlyPatterns.some((pattern) =>
+      pattern.test(`https://${hostname}`)
+    );
+  }
+
+  return passThroughPatterns.some((pattern) =>
+    pattern.test(`https://${hostname}`)
+  );
+}

test/http-combo-server.spec.ts renamed to test/server-utils.spec.ts

@@ -1,6 +1,6 @@
 import { URLPattern } from "urlpattern-polyfill";
-import { shouldPassThrough } from "../src/server/http-combo-server";
 import { expect } from "./test-utils";
+import { shouldPassThrough } from "../src/util/server-utils";
 
 describe("shouldPassThrough", () => {
   it("should return false when passThroughHostnames is empty and interceptOnlyHostnames is undefined", async () => {