FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

feat: add api key support by arithmetic1728 · Pull Request #969 · googleapis/gapic-generator-python · GitHub

This repository was archived by the owner on Mar 26, 2026. It is now read-only.

feat: add api key support - #969

Merged
arithmetic1728 merged 3 commits into
masterfrom
apikey
Jan 19, 2022
Merged

feat: add api key support#969
arithmetic1728 merged 3 commits into
masterfrom
apikey

Conversation

arithmetic1728 commented Aug 11, 2021
edited
Loading

Copy link
Copy Markdown
Contributor

This PR adds API key support. Internal doc: go/gapic-api-key-support

If api_key is provided via client_options, use the api key. Note that GOOGLE_API_KEY is handled by google-auth so we don't need to do anything in microgenerator.

Related PRs:
googleapis/google-auth-library-python#826
googleapis/python-api-core#248

Tested with language api:
https://github.com/arithmetic1728/python-language/commit/da9b1215d1a46e5dc2ccc10bf9a6d1edd9cb8940

google-cla Bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 11, 2021
arithmetic1728 added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Sep 8, 2021
arithmetic1728 removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jan 19, 2022
arithmetic1728 marked this pull request as ready for review January 19, 2022 08:54
arithmetic1728 requested review from a team January 19, 2022 08:54
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL