Skip to content
This repository has been archived by the owner on Sep 20, 2023. It is now read-only.

fix(v1beta2): enable self signed jwt for grpc #191

Merged
merged 7 commits into from Sep 1, 2021

Conversation

nicain
Copy link
Contributor

@nicain nicain commented Aug 10, 2021

Remove Owlbot v1beta2 customization that passes default scopes as user scopes, so that the auth library does not use the self-signed JWT flow.

Fixes #174

…s user scopes, so that the auth library does not use the self-signed JWT flow.
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 10, 2021
@product-auto-label product-auto-label bot added the api: documentai Issues related to the googleapis/python-documentai API. label Aug 10, 2021
@nicain nicain requested a review from busunkim96 August 10, 2021 18:49
@nicain
Copy link
Contributor Author

nicain commented Aug 10, 2021

@busunkim96 Are there special Labels necessary to engage owlbot for this change?

@google-cla
Copy link

google-cla bot commented Aug 12, 2021

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@google-cla google-cla bot added cla: no This human has *not* signed the Contributor License Agreement. and removed cla: yes This human has signed the Contributor License Agreement. labels Aug 12, 2021
@parthea
Copy link
Contributor

parthea commented Aug 12, 2021

@nicain I'm not aware of any labels. I ran owlbot locally using these steps and pushed to this branch.

# pull the latest docker image for the owlbot-cli
docker pull gcr.io/repo-automation-bots/owlbot-cli:latest
# run the owlbot-cli container to pull changes from googleapis/googleapis-gen
docker run --rm --user $(id -u):$(id -g) -v $(pwd):/repo -w /repo gcr.io/repo-automation-bots/owlbot-cli:latest copy-code
# pull the latest python post processing image
docker pull gcr.io/repo-automation-bots/owlbot-python:latest
# run the python post processing container to synchronize templated files and format the code
docker run --user $(id -u):$(id -g) --rm -v $(pwd):/repo -w /repo gcr.io/repo-automation-bots/owlbot-python:latest

@googlebot I consent.

@google-cla
Copy link

google-cla bot commented Aug 12, 2021

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@parthea
Copy link
Contributor

parthea commented Aug 12, 2021

@googlebot I consent.

@google-cla google-cla bot added cla: yes This human has signed the Contributor License Agreement. and removed cla: no This human has *not* signed the Contributor License Agreement. labels Aug 12, 2021
owlbot.py Outdated Show resolved Hide resolved
@busunkim96 busunkim96 mentioned this pull request Aug 12, 2021
@parthea parthea marked this pull request as ready for review September 1, 2021 11:16
@parthea parthea requested a review from a team as a code owner September 1, 2021 11:16
@parthea parthea changed the title fix: Remove Owlbot v1beta2 customization fix(v1beta2): enable self signed jwt for grpc Sep 1, 2021
@parthea parthea added the automerge Merge the pull request once unit tests and other checks pass. label Sep 1, 2021
@parthea parthea merged commit 14e7765 into googleapis:main Sep 1, 2021
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Sep 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: documentai Issues related to the googleapis/python-documentai API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enable self-signed JWT flow for v1beta2
3 participants