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: Support GOOGLE_CLOUD_UNIVERSE_DOMAIN environment variable. #2913

Merged
merged 1 commit into from
Jan 15, 2025

Conversation

amanda-tarafa
Copy link
Contributor

Towards b/325308829

@amanda-tarafa amanda-tarafa requested a review from jskeet January 14, 2025 22:01
@amanda-tarafa amanda-tarafa requested a review from a team as a code owner January 14, 2025 22:01
@amanda-tarafa amanda-tarafa merged commit b0b072b into googleapis:main Jan 15, 2025
3 checks passed
@amanda-tarafa amanda-tarafa deleted the universe-env-var branch January 15, 2025 08:28
amanda-tarafa added a commit to amanda-tarafa/google-api-dotnet-client that referenced this pull request Jan 23, 2025
Fixes:

- googleapis#2869 Use universe-domain instead of universe_domain as the MDS endpoint
- googleapis#2870 BaseClientService.UniverDomain setter is obsolete
- googleapis#2871 Pause automatic requests to MDS Universe Domain endpoint

Features:

- googleapis#2746 Simplify setting the HttpClient timeout
- googleapis#2809 Improve error handling when signing with the IAM service

  BREAKING CHANGE: The ComputeCredential and ImpersonatedCredential SignBlobAsync methods will throw a GoogleApiException instead of a HttpRequestExtension. The GoogleApiException makes the HttpResponseMessage content available, which usually includes details about the error.
  We consider the risk of this change breaking users  lower than the risk of disrupting all users with a new major version so we've decided to release this breaking change on a minor version of the library. Please create an issue on this repo if you are affected and we will e happy to help.

- googleapis#2879 Use recommended retries for token and IAM sign blob endpoints
- googleapis#2913 Support GOOGLE_CLOUD_UNIVERSE_DOMAIN environment variable

Dependencies:

- googleapis#2730 Remove unused dependency Microsoft.AspNetCore.Authorization from Google.Apis.Auth.AspNetCore3

  BREAKING CHANGE: Projects using Google.Apis.Auth.AspNetCore3 that transitively depend on Microsoft.AspNetCore.Authorization may be broken. They only need to add an implicit dependency themselves. We consider the risk of this change breaking users  lower than the risk of disrupting all users with a new major version so we've decided to release this breaking change on a minor version of the library. Please create an issue on this repo if you are affected and we will e happy to help.

Documentation:

- googleapis#2916 Add warning note about user provided credential configurations
amanda-tarafa added a commit that referenced this pull request Jan 23, 2025
Fixes:

- #2869 Use universe-domain instead of universe_domain as the MDS endpoint
- #2870 BaseClientService.UniverDomain setter is obsolete
- #2871 Pause automatic requests to MDS Universe Domain endpoint

Features:

- #2746 Simplify setting the HttpClient timeout
- #2809 Improve error handling when signing with the IAM service

  BREAKING CHANGE: The ComputeCredential and ImpersonatedCredential SignBlobAsync methods will throw a GoogleApiException instead of a HttpRequestExtension. The GoogleApiException makes the HttpResponseMessage content available, which usually includes details about the error.
  We consider the risk of this change breaking users  lower than the risk of disrupting all users with a new major version so we've decided to release this breaking change on a minor version of the library. Please create an issue on this repo if you are affected and we will e happy to help.

- #2879 Use recommended retries for token and IAM sign blob endpoints
- #2913 Support GOOGLE_CLOUD_UNIVERSE_DOMAIN environment variable

Dependencies:

- #2730 Remove unused dependency Microsoft.AspNetCore.Authorization from Google.Apis.Auth.AspNetCore3

  BREAKING CHANGE: Projects using Google.Apis.Auth.AspNetCore3 that transitively depend on Microsoft.AspNetCore.Authorization may be broken. They only need to add an implicit dependency themselves. We consider the risk of this change breaking users  lower than the risk of disrupting all users with a new major version so we've decided to release this breaking change on a minor version of the library. Please create an issue on this repo if you are affected and we will e happy to help.

Documentation:

- #2916 Add warning note about user provided credential configurations
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants