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: return a friendly error if the dialer is closed #538

Merged
merged 1 commit into from
Apr 9, 2024
Merged

Conversation

enocom
Copy link
Member

@enocom enocom commented Apr 3, 2024

If the dialer has already been closed, return a clear error.

Fixes #522

@enocom
Copy link
Member Author

enocom commented Apr 3, 2024

cc @mvdan in case you'd like to provide any feedback.

@enocom enocom force-pushed the friendly-error branch 3 times, most recently from f2f1623 to e9d41d3 Compare April 3, 2024 18:24
Copy link

@mvdan mvdan left a comment

Choose a reason for hiding this comment

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

Thanks, SGTM! Just a thought about the error message itself.

dialer.go Outdated Show resolved Hide resolved
If the dialer has already been closed, return a clear error.

Fixes #522
Copy link
Collaborator

@jackwotherspoon jackwotherspoon left a comment

Choose a reason for hiding this comment

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

LGTM ✅

@enocom enocom merged commit 66d7bd0 into main Apr 9, 2024
16 checks passed
@enocom enocom deleted the friendly-error branch April 9, 2024 00:15
enocom added a commit to GoogleCloudPlatform/cloud-sql-go-connector that referenced this pull request Apr 9, 2024
enocom added a commit to GoogleCloudPlatform/cloud-sql-go-connector that referenced this pull request Apr 9, 2024
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.

Occasional "context cancelled" dial errors even though we don't use cancellable contexts
3 participants