Fix query params and regional routing in emulated HTTP functions (#3033)

Author: samtstern

CHANGELOG.md

@@ -1,2 +1,4 @@
 - Catches errors while updating authorized domains when deleting channels, printing a warning instead of failing.
 - Fixes issue where `host` header was being incorrectly set when proxying to Cloud Run or Cloud Functions for Firebase from the Hosting emulator. (#3012)
+- Fixes issue where emulated HTTP functions would crash when the URL contained query parameters (#3032)
+- Fixes issue with routing to emulated HTTP functions in regions outside of `us-central1` (#3031)

src/emulator/functionsEmulator.ts

@@ -1068,8 +1068,8 @@ export class FunctionsEmulator implements EmulatorInstance {
     // req.url = /:projectId/:region/:trigger_name/*
     const url = new URL(`${req.protocol}://${req.hostname}${req.url}`);
     const path = `${url.pathname}${url.search}`.replace(
-      `/${this.args.projectId}/us-central1/${triggerId}`,
-      ""
+      new RegExp(`\/${this.args.projectId}\/[^\/]*\/${triggerId}\/?`),
+      "/"
     );
 
     // We do this instead of just 302'ing because many HTTP clients don't respect 302s so it may

src/test/emulators/functionsEmulator.spec.ts

@@ -227,6 +227,34 @@ describe("FunctionsEmulator-Hub", () => {
       });
   }).timeout(TIMEOUT_LONG);
 
+  it("should return the correct url, baseUrl, originalUrl with query params", async () => {
+    useFunctions(() => {
+      require("firebase-admin").initializeApp();
+      return {
+        function_id: require("firebase-functions").https.onRequest(
+          (req: express.Request, res: express.Response) => {
+            res.json({
+              url: req.url,
+              baseUrl: req.baseUrl,
+              originalUrl: req.originalUrl,
+              query: req.query,
+            });
+          }
+        ),
+      };
+    });
+
+    await supertest(functionsEmulator.createHubServer())
+      .get("/fake-project-id/us-central1/function_id?a=1&b=2")
+      .expect(200)
+      .then((res) => {
+        expect(res.body.url).to.eq("/?a=1&b=2");
+        expect(res.body.baseUrl).to.eq("");
+        expect(res.body.originalUrl).to.eq("/?a=1&b=2");
+        expect(res.body.query).to.deep.eq({ a: "1", b: "2" });
+      });
+  }).timeout(TIMEOUT_LONG);
+
   it("should return the correct url, baseUrl, originalUrl for a subroute", async () => {
     useFunctions(() => {
       require("firebase-admin").initializeApp();
@@ -253,6 +281,32 @@ describe("FunctionsEmulator-Hub", () => {
       });
   }).timeout(TIMEOUT_LONG);
 
+  it("should return the correct url, baseUrl, originalUrl for any region", async () => {
+    useFunctions(() => {
+      require("firebase-admin").initializeApp();
+      return {
+        function_id: require("firebase-functions")
+          .region("europe-west3")
+          .https.onRequest((req: express.Request, res: express.Response) => {
+            res.json({
+              url: req.url,
+              baseUrl: req.baseUrl,
+              originalUrl: req.originalUrl,
+            });
+          }),
+      };
+    });
+
+    await supertest(functionsEmulator.createHubServer())
+      .get("/fake-project-id/europe-west3/function_id")
+      .expect(200)
+      .then((res) => {
+        expect(res.body.url).to.eq("/");
+        expect(res.body.baseUrl).to.eq("");
+        expect(res.body.originalUrl).to.eq("/");
+      });
+  }).timeout(TIMEOUT_LONG);
+
   it("should route request body", async () => {
     useFunctions(() => {
       require("firebase-admin").initializeApp();