-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Actually check connection errors #4465
Conversation
21786b4
to
6252f17
Compare
With actual error checking, a helpful response is returned:
|
Nice catch, much improved! |
@@ -158,7 +158,10 @@ Examples: | |||
} | |||
|
|||
if err := c.connect(""); err != nil { | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually do something if an error is returned, don't just carry on.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what??? lol, crazy! I think code reviews failed us the last time. Nice catch.
Fix issue #4463
6252f17
to
81390db
Compare
👍 |
2 similar comments
👍 |
+1 |
Actually check connection errors
Fix issue #4463