Google-cloud-go: firestore: detect-project-id overrides the emulator

Created on 27 Jan 2020 · 4Comments · Source: googleapis/google-cloud-go

Client

Firestore

Environment

gcloud cloud-firestore-emulator 1.10.2 running with command
gcloud beta emulators firestore start --host-port "localhost:8081"
Linux runtime environment, bash shell with
export FIRESTORE_EMULATOR_HOST=localhost:8081
latest golang firestore server client

Code

        client, err = firestore.NewClient(context.Background(),
            "*detect-project-id*",
            options...)                                                                                                                                                                  
        if err != nil {
            return err 
        }

Expected behavior

By default, the client lib ignores the specified Firestore project ID in a NewClient() call when it detects an emulator is running (i.e. FIRESTORE_EMULATOR_HOST env var is set) ... unless the project ID is set to the "*detect-project-id*" sentinel value. It should check for the emulator env var before checking for the "*detect-project-id*" project ID sentinel value, and ignore this value in the same way as it ignores any other project ID value. Developers using the sentinel value to auto-detect projects when running their code in GCP should not need to modify their code/configuration to use a different project ID value when using the local emulator. (I'm guessing that was the intended behavior, since it is ignored for every other value.)

Actual behavior

When the sentinel value is used, the attempt to instantiate the client throws the following error:
firestore: see the docs on DetectProjectID

Screenshots

N/A

Additional context

None

firestore p2 bug

Source