add support for wildcards in shouldPassThrough

Author: lipsumar

package.json

@@ -197,6 +197,7 @@
     "semver": "^7.5.3",
     "socks-proxy-agent": "^7.0.0",
     "typed-error": "^3.0.2",
+    "urlpattern-polyfill": "^10.0.0",
     "uuid": "^8.3.2",
     "ws": "^8.8.0"
   }

src/server/http-combo-server.ts

@@ -14,6 +14,7 @@ import {
     NonTlsError,
     readTlsClientHello
 } from 'read-tls-client-hello';
+import { URLPattern } from "urlpattern-polyfill";
 
 import { TlsHandshakeFailure } from '../types';
 import { getCA } from '../util/tls';
@@ -426,12 +427,16 @@ export function shouldPassThrough(
     // Only one of these two should have values (validated above):
     passThroughHostnames: string[],
     interceptOnlyHostnames: string[] | undefined
-): boolean {
+  ): boolean {
     if (!hostname) return false;
-
+  
     if (interceptOnlyHostnames) {
-        return !interceptOnlyHostnames.includes(hostname);
+      return !interceptOnlyHostnames.some((hn) =>
+        new URLPattern(`https://${hn}`).test(`https://${hostname}`)
+      );
     }
-
-    return passThroughHostnames.includes(hostname);
-}
+  
+    return passThroughHostnames.some((hn) =>
+      new URLPattern(`https://${hn}`).test(`https://${hostname}`)
+    );
+  }

test/http-combo-server.spec.ts

@@ -35,6 +35,11 @@ describe("shouldPassThrough", () => {
       );
       expect(should).to.be.false;
     });
+
+    it("should return true when hostname match a wildcard", () => {
+      const should = shouldPassThrough("example.org", ["*.org"], undefined);
+      expect(should).to.be.true;
+    });
   });
   describe("interceptOnlyHostnames", () => {
     it("should return false when hostname is in interceptOnlyHostnames", () => {
@@ -46,5 +51,10 @@ describe("shouldPassThrough", () => {
       const should = shouldPassThrough("example.org", [], ["example.com"]);
       expect(should).to.be.true;
     });
+
+    it("should return false when hostname match a wildcard", () => {
+      const should = shouldPassThrough("example.org", [], ["*.org"]);
+      expect(should).to.be.false;
+    });
   });
 });