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

feat(auth): add downscope package #8532

Merged
merged 4 commits into from
Sep 11, 2023
Merged

feat(auth): add downscope package #8532

merged 4 commits into from
Sep 11, 2023

Conversation

codyoss
Copy link
Member

@codyoss codyoss commented Sep 6, 2023

This is roughly the equivalent of the old oauth2/google/downscope package. Some integration tests are being added here but disabling them until a future commit where all test infra will be updated to support these tests. Ran them locally for now and all seems well.

This is roughtly the equilvalent of the old oauth2/google/downscope
package. Some integration tests are being added here but disabling
them until a future commit where all test infra will be updated to
support these tests. Ran them locally for now and all seems well.
@codyoss codyoss requested a review from a team as a code owner September 6, 2023 19:22
@product-auto-label product-auto-label bot added the size: l Pull request size is large. label Sep 6, 2023
Copy link
Member

@quartzmo quartzmo left a comment

Choose a reason for hiding this comment

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

LGTM

@codyoss codyoss enabled auto-merge (squash) September 6, 2023 20:53
@codyoss codyoss disabled auto-merge September 11, 2023 15:42
@codyoss codyoss added the automerge Merge the pull request once unit tests and other checks pass. label Sep 11, 2023
@codyoss codyoss merged commit dda9bff into googleapis:main Sep 11, 2023
@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 11, 2023
@codyoss codyoss deleted the auth4 branch September 11, 2023 15:44
This was referenced Sep 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: l Pull request size is large.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants