Skip to content

Conversation

busunkim96
Copy link
Contributor

@busunkim96 busunkim96 commented Jan 20, 2021

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 google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Jan 20, 2021
@busunkim96 busunkim96 marked this pull request as ready for review February 3, 2021 17:38
@busunkim96 busunkim96 requested a review from a team as a code owner February 3, 2021 17:38
@busunkim96 busunkim96 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 3, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 3, 2021
@busunkim96 busunkim96 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 4, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 4, 2021
@busunkim96
Copy link
Contributor Author

Reviewers, PTAL. Thank you!

Copy link
Contributor

@arithmetic1728 arithmetic1728 left a comment

Choose a reason for hiding this comment

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

LGTM

@busunkim96 busunkim96 merged commit 94c76e0 into master Feb 5, 2021
@busunkim96 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 <[email protected]>
This was referenced May 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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