Skip to content
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

fix FIREBASE_DATABASE_EMULATOR_HOST not working #578

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

timesking
Copy link

Like #574 mentioned.
FIREBASE_DATABASE_EMULATOR_HOST is not working at all.

the db URL should be http://127.0.0.1:9000?ns=dbname or https://.firebaseio.com

The explanation of FIREBASE_DATABASE_EMULATOR_HOST in doc https://firebase.google.com/docs/emulator-suite/connect_rtdb#admin_sdks, says Format should only be host:port, "127.0.0.1:9000".

With this modification:

When you have FIREBASE_DATABASE_EMULATOR_HOST: 127.0.0.1:9000 in Env
You could set your DBUrl to either http://127.0.0.1:9000?ns=dbname or https://.firebaseio.com.

The Go SDK will work as other languages firebase SDK.

…ame>.firebaseio.com

	// FIREBASE_DATABASE_EMULATOR_HOST in doc https://firebase.google.com/docs/emulator-suite/connect_rtdb#admin_sdks
	// Format should only be host:port, "127.0.0.1:9000"
@lahirumaramba
Copy link
Member

lahirumaramba commented Sep 15, 2023

Thank you for your contribution! Please change the base branch to dev before we can start the review process.

https://github.com/firebase/firebase-admin-go/blob/master/CONTRIBUTING.md#want-to-submit-a-pull-request

Make sure to create all your pull requests against the dev branch. All development work takes place on this branch, while the master branch is dedicated for released stable code. This enables us to review and merge routine code changes, without impacting downstream applications that are building against our master branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants