-
Notifications
You must be signed in to change notification settings - Fork 901
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Outdated allowed SSR region #6223
Comments
This issue does not seem to follow the issue template. Make sure you provide all the required information. |
There is a PR created to solve this issue at #6086. |
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. |
@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. |
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() { |
#6086 was merged, can this issue be closed? |
[REQUIRED] Environment info
firebase-tools: v12.4.7
Platform: macOS
[REQUIRED] Test case
firebase.json
[REQUIRED] Steps to reproduce
Set firebase hosting config as above.
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.
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
The text was updated successfully, but these errors were encountered: