Skip to content
This repository was archived by the owner on May 12, 2026. It is now read-only.

feat: adds universe domain for DownscopedCredentials and ExternalAccountAuthorizedUserCredentials - #1355

Merged
lsirac merged 11 commits into
googleapis:mainfrom
lsirac:downscoped
Jan 25, 2024
Merged

lsirac merged 11 commits into
googleapis:mainfrom
lsirac:downscoped

Conversation

@lsirac

@lsirac lsirac commented Jan 18, 2024

Copy link
Copy Markdown
Contributor
  • Adds universe domain support for DownscopedCredentials.
  • Adds universe domain support for ExternalAccountAuthorizedUserCredentials.
  • Adds more tests for universe domain in the BYOID credential classes.

@lsirac
lsirac requested review from a team January 18, 2024 00:48
@product-auto-label product-auto-label Bot added the size: l Pull request size is large. label Jan 18, 2024
@lsirac lsirac added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jan 18, 2024
Comment thread oauth2_http/java/com/google/auth/oauth2/DownscopedCredentials.java
@TimurSadykov
TimurSadykov self-requested a review January 20, 2024 00:54
@TimurSadykov

Copy link
Copy Markdown

looking

Comment thread oauth2_http/java/com/google/auth/oauth2/DownscopedCredentials.java
Comment thread oauth2_http/java/com/google/auth/oauth2/DownscopedCredentials.java
Comment thread oauth2_http/java/com/google/auth/oauth2/DownscopedCredentials.java
Comment thread oauth2_http/java/com/google/auth/oauth2/DownscopedCredentials.java

@TimurSadykov TimurSadykov left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM, one test suggestions

@lsirac lsirac removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jan 24, 2024
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@lsirac
lsirac merged commit 17ef707 into googleapis:main Jan 25, 2024
@lsirac
lsirac deleted the downscoped branch January 25, 2024 18:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size: l Pull request size is large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants