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

fix(transport): relax universe checks #2376

Merged

Conversation

shollyman
Copy link
Contributor

This PR serves to relax universe mismatch checks, as a user specifying credentials via the option.WithTokenSource has provided a form of credential where the universe information is not directly accessible.

In these cases, we no longer perform the universe mismatch checks.

This PR also augments the existing mismatch checks to exercise a more diverse set of client options.

This PR serves to relax universe mismatch checks, as a user specifying
credentials via the option.WithTokenSource has provided a form of
credential where the universe information is not directly accessible.

In these cases, we do not perform the universe mismatch checks.

This PR also augments the existing mismatch checks to exercise a more
diverse set of client options.
@shollyman shollyman requested a review from a team as a code owner January 26, 2024 01:18
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.

Thank you!

@shollyman shollyman added the automerge Merge the pull request once unit tests and other checks pass. label Jan 26, 2024
@gcf-merge-on-green gcf-merge-on-green bot merged commit 55b0516 into googleapis:main Jan 26, 2024
5 checks passed
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Jan 26, 2024
@shollyman shollyman deleted the relax-tokensource-checks branch January 26, 2024 16:57
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.

None yet

2 participants