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

Use project ID specified in config for Pubsub client #243

Merged
merged 1 commit into from
Mar 26, 2024

Conversation

h-michael
Copy link
Contributor

Close #240

If the Pubsub client tries to access the pubsub topic or subscription of a project different from the service account's project, the TokenSource's project_id (the client's service account's project ID) will be set, so the topic and subscription of the external project will be set. cannot be accessed.
Therefore, if the project ID is specified in the config, use that.

If the Pubsub client tries to access the pubsub topic or subscription of a
project different from the service account's project, the TokenSource's
project_id (the client's service account's project ID) will be set, so the
topic and subscription of the external project will be set. cannot be
accessed.
Therefore, if the project ID is specified in the config, use that.
@yoshidan yoshidan added the safe to test safe to test label Mar 24, 2024
@yoshidan yoshidan merged commit e0fe8e9 into yoshidan:main Mar 26, 2024
6 checks passed
@h-michael h-michael deleted the do-not-overwrite-project-id branch March 26, 2024 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
safe to test safe to test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Project ID is overwritten with service account's project id even if another project id configured.
2 participants