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(auth): restore use of grpc.Dial #11118

Merged
merged 2 commits into from
Nov 12, 2024
Merged

Conversation

codyoss
Copy link
Member

@codyoss codyoss commented Nov 12, 2024

There have been a couple of reports lately around some subtle changes in behaviour of using the new recommend method. There is also a known bug, see grpc/grpc-go#7556. Until the bug is fixed and we have a way forward we will revert to calling the deprecated Dial function.

Fixes: #10780

There have been a couple of reports lately around some subtle
changes in behaviour of using the new recommend method. There is
also a known bug, see grpc/grpc-go#7556.
Until the bug is fixed and we have a way forward we will revert to
calling the deprecated Dial function.

Fixes: googleapis#7556
@codyoss codyoss requested a review from a team as a code owner November 12, 2024 20:25
@quartzmo
Copy link
Member

Note that this reverts changes in #10780.

@codyoss codyoss added the automerge Merge the pull request once unit tests and other checks pass. label Nov 12, 2024
@gcf-merge-on-green gcf-merge-on-green bot merged commit 2456b94 into googleapis:main Nov 12, 2024
8 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 Nov 12, 2024
@codyoss codyoss deleted the grpc branch November 12, 2024 20:45
tritone added a commit to tritone/google-cloud-go that referenced this pull request Nov 14, 2024
This picks up googleapis#11118 which should fix dial hangs for storage gRPC
methods.
tritone added a commit that referenced this pull request Nov 14, 2024
This picks up #11118 which should fix dial hangs for storage gRPC
methods.
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