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

feat: allow default_host and default_scopes to be passed to create_channel by busunkim96 · Pull Request #134 · googleapis/python-api-core · GitHub

This repository was archived by the owner on Feb 23, 2026. It is now read-only.
/ python-api-core Public archive

feat: allow default_host and default_scopes to be passed to create_channel - #134

Merged
busunkim96 merged 12 commits into
masterfrom
self-signed-jwt
Feb 5, 2021
Merged

feat: allow default_host and default_scopes to be passed to create_channel#134
busunkim96 merged 12 commits into
masterfrom
self-signed-jwt

Conversation

busunkim96 commented Jan 20, 2021
edited
Loading

Copy link
Copy Markdown
Contributor

Add default_host and default_scopes parameters to create_channel so self-signed JWTs can be used. See googleapis/google-auth-library-python#665 for the change in the auth library and googleapis/python-translate#107 for the changes in the GAPIC library.

google-cla Bot added the cla: yes This human has signed the Contributor License Agreement. label Jan 20, 2021
busunkim96 marked this pull request as ready for review February 3, 2021 17:38
busunkim96 requested review from a team and arithmetic1728 February 3, 2021 17:38
busunkim96 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 3, 2021
yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 3, 2021
Comment thread google/api_core/grpc_helpers.py Outdated
Comment thread noxfile.py Outdated
busunkim96 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 4, 2021
yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 4, 2021

Copy link
Copy Markdown
Contributor Author

Reviewers, PTAL. Thank you!

arithmetic1728 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM

busunkim96 merged commit 94c76e0 into master Feb 5, 2021
busunkim96 deleted the self-signed-jwt branch February 5, 2021 17:26
gcf-merge-on-green Bot pushed a commit to googleapis/google-auth-library-python that referenced this pull request Mar 16, 2021
Move the version from `setup.py` to `google/auth/version.py`.  Same as googleapis/python-api-core#80.  (see googleapis/python-api-core#27 for motivation). This is option 3 in https://packaging.python.org/guides/single-sourcing-package-version/.

This unblocks a version check I'd like to add in googleapis/python-api-core#134.

Usage:

```py
>>> import google.auth
>>> google.auth.__version__
'1.25.0'
```
busunkim96 added a commit to googleapis/gapic-generator-python that referenced this pull request Apr 21, 2021
See [RFC (internal only)](https://docs.google.com/document/d/1SNCVTmW6Rtr__u-_V7nsT9PhSzjj1z0P9fAD3YUgRoc/edit#) and https://aip.dev/auth/4111.

Support the self-signed JWT flow for service accounts by passing `default_scopes` and `default_host` in calls to the auth library and `create_channel`. This depends on features exposed in the following PRs: googleapis/python-api-core#134, googleapis/google-auth-library-python#665.

It may be easier to look at https://github.com/googleapis/python-translate/pull/107/files for a diff on a real library.

This change is written so that the library is (temporarily) compatible with older `google-api-core` and `google-auth` versions. Because of this it not possible to reach 100% coverage on a single unit test run. `pytest` runs twice in two of the `nox` sessions.

Miscellaneous changes:
- sprinkled in `__init__.py` files in subdirs of the `test/` directory, as otherwise pytest-cov seems to fail to collect coverage properly in some instances.
- new dependency on `packaging` for Version comparison https://pypi.org/project/packaging/

Co-authored-by: Brent Shaffer <betterbrent@google.com>
This was referenced May 30, 2025
parthea pushed a commit to googleapis/google-cloud-python that referenced this pull request Nov 24, 2025
See [RFC (internal only)](https://docs.google.com/document/d/1SNCVTmW6Rtr__u-_V7nsT9PhSzjj1z0P9fAD3YUgRoc/edit#) and https://aip.dev/auth/4111.

Support the self-signed JWT flow for service accounts by passing `default_scopes` and `default_host` in calls to the auth library and `create_channel`. This depends on features exposed in the following PRs: googleapis/python-api-core#134, googleapis/google-auth-library-python#665.

It may be easier to look at https://github.com/googleapis/python-translate/pull/107/files for a diff on a real library.

This change is written so that the library is (temporarily) compatible with older `google-api-core` and `google-auth` versions. Because of this it not possible to reach 100% coverage on a single unit test run. `pytest` runs twice in two of the `nox` sessions.

Miscellaneous changes:
- sprinkled in `__init__.py` files in subdirs of the `test/` directory, as otherwise pytest-cov seems to fail to collect coverage properly in some instances.
- new dependency on `packaging` for Version comparison https://pypi.org/project/packaging/

Co-authored-by: Brent Shaffer <betterbrent@google.com>
parthea pushed a commit to googleapis/google-cloud-python that referenced this pull request Nov 26, 2025
Move the version from `setup.py` to `google/auth/version.py`.  Same as googleapis/python-api-core#80.  (see googleapis/python-api-core#27 for motivation). This is option 3 in https://packaging.python.org/guides/single-sourcing-package-version/.

This unblocks a version check I'd like to add in googleapis/python-api-core#134.

Usage:

```py
>>> import google.auth
>>> google.auth.__version__
'1.25.0'
```
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.

4 participants


Back | FazBrowse Home | New Git URL