Outdated allowed SSR region

[eXodes]

[REQUIRED] Environment info

firebase-tools: v12.4.7

Platform: macOS

[REQUIRED] Test case

firebase.json

{
  // ...
  "hosting": {
      "source": ".",
      "ignore": ["firebase.json", "**/.*", "**/node_modules/**"],
      "frameworksBackend": {
          "region": "asia-southeast1",
          "maxInstances": 1
      }
  },
  // ...
}

[REQUIRED] Steps to reproduce

1. Set firebase hosting config as above.

2. Deploy firebase hosting with experiments env

FIREBASE_CLI_EXPERIMENTS="webframeworks,pintags" firebase deploy --only hosting

[REQUIRED] Expected behavior

Functions to be deployed successfully since asia-southeast1 supports 2nd gen functions.

[REQUIRED] Actual behavior

Error during deployment due to unsupported allowed SSR region.

  Error: Hosting config for site web-app places server-side content in region asia-southeast1 which is not known. Valid regions are us-central1, us-west1, us-east1, europe-west1, and asia-east1

---

The initial motive of this implementation is stated on this PR #5504. The constant has been refactored into frameworks/constants.ts. However, currently there are more regions that supports 2nd gen functions.

Reference: Cloud Functions supported regions

[google-oss-bot]

This issue does not seem to follow the issue template. Make sure you provide all the required information.

[eXodes]

There is a PR created to solve this issue at #6086.

[aalej]

Hi @eXodes, thanks for reporting this issue and providing details about your observations. I was able to reproduce the error you mentioned. Let me inform our team about this so that they can look into the existing PR and try to bring someone here who can provide more context about this.

[al4nnw]

@aalej, any updates on this matter? The region distance is causing load time issues for of our users. It's very important to address this, especially since V2 functions are already compatible with other regions.

[mfellner]

Meanwhile you can use patch-package with a diff like this (worked for me):

diff --git a/node_modules/firebase-tools/lib/frameworks/constants.js b/node_modules/firebase-tools/lib/frameworks/constants.js
index a05d65c..30f6600 100644
--- a/node_modules/firebase-tools/lib/frameworks/constants.js
+++ b/node_modules/firebase-tools/lib/frameworks/constants.js
@@ -28,6 +28,7 @@ exports.ALLOWED_SSR_REGIONS = [
     { name: "us-east1 (South Carolina)", value: "us-east1" },
     { name: "europe-west1 (Belgium)", value: "europe-west1" },
     { name: "asia-east1 (Taiwan)", value: "asia-east1" },
+    { name: "europe-west3 (Frankfurt)", value: "europe-west3" },
 ];
 exports.I18N_ROOT = "/";
 function GET_DEFAULT_BUILD_TARGETS() {

[mfellner]

#6086 was merged, can this issue be closed?

[aalej]

Thanks @mfellner, seems like this issue should be fixed in v.13.0.1. I'm going to go ahead and close this.