Skip to content

feat: adds universe domain for DownscopedCredentials and ExternalAccountAuthorizedUserCredentials#1355

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

feat: adds universe domain for DownscopedCredentials and ExternalAccountAuthorizedUserCredentials#1355
lsirac merged 11 commits intogoogleapis:mainfrom
lsirac:downscoped

Conversation

@lsirac
Copy link
Copy Markdown
Contributor

@lsirac lsirac commented Jan 18, 2024

  • 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
Copy link
Copy Markdown

@TimurSadykov TimurSadykov left a comment

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 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.

3 participants