Skip to content

feat: updates UserAuthorizer to support retrieving token response directly with different client auth types #1486

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.

Sign up for GitHub

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

Jump to bottom
Merged
lsirac merged 8 commits into from
Sep 5, 2024
Merged

feat: updates UserAuthorizer to support retrieving token response directly with different client auth types #1486

lsirac merged 8 commits into from
Sep 5, 2024

Conversation

@lsirac
Copy link
Contributor

@lsirac lsirac commented Aug 28, 2024 •
edited
Loading

Adds support for the following in UserAuthorizer:

  • Different client auth types
  • A method to retrieve the token response directly, not UserCredentials, allowing for it to be used in more ways (e.g. enables Workforce Identity Federation)
  • Constructor now takes in the Builder directly
@lsirac lsirac requested a review from a team as a code owner August 28, 2024 19:39
@product-auto-label product-auto-label bot added the size: xl Pull request size is extra large. label Aug 28, 2024
@sonarqubecloud
Copy link

sonarqubecloud bot commented Aug 28, 2024

Quality Gate Passed Quality Gate passed

Issues
3 New issues
0 Accepted issues

Measures
0 Security Hotspots
97.4% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud
@lsirac lsirac requested a review from aeitzman August 28, 2024 19:57
aeitzman
aeitzman approved these changes Aug 28, 2024
View reviewed changes
@lsirac lsirac requested a review from a team as a code owner September 3, 2024 22:12
@lsirac lsirac requested a review from lqiu96 September 3, 2024 22:17
@lqiu96 lqiu96 requested a review from zhumin8 September 4, 2024 14:29
lqiu96
lqiu96 reviewed Sep 4, 2024
View reviewed changes
lqiu96
lqiu96 reviewed Sep 4, 2024
View reviewed changes
sai-sunder-s
sai-sunder-s approved these changes Sep 5, 2024
View reviewed changes
@lsirac lsirac requested a review from lqiu96 September 5, 2024 10:28
@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 5, 2024

Quality Gate Passed Quality Gate passed

Issues
4 New issues
0 Accepted issues

Measures
0 Security Hotspots
98.6% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud
@lqiu96
Copy link
Member

lqiu96 commented Sep 5, 2024

Thanks @lsirac, LGTM. Added some last comment if you could take a look as well.
@lsirac lsirac merged commit 1651006 into googleapis:main Sep 5, 2024
@release-please release-please bot mentioned this pull request Sep 5, 2024
Merged
@release-please release-please bot mentioned this pull request Dec 11, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: xl Pull request size is extra large.

4 participants

@lsirac @lqiu96 @sai-sunder-s @aeitzman