-
Notifications
You must be signed in to change notification settings - Fork 166
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
Provide more information for socket errors #58
Comments
5 tasks
gboer
added a commit
to gboer/couchdb-nano
that referenced
this issue
Dec 9, 2021
…to support easier debugging of connection issues (issue apache#58)
gboer
added a commit
to gboer/couchdb-nano
that referenced
this issue
Dec 9, 2021
…when connecting, to support easier debugging of connection issues
glynnbird
pushed a commit
that referenced
this issue
Dec 21, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
It would be helpful to provide additional error information in the message Nano outputs for socket or connection errors.
Expected Behavior
Suggest including, for example, reading the error
.code
and.description
and adding it to the message to provide some more useful diagnostics.err.message ->
error happened in your connection ECONNRESET socket hang up
Current Behavior
Currently for a socket/connection error only the
err.message
is set toerror happened in your connection
.err.message ->
error happened in your connection
Possible Solution
Could build up the error message to include
.code
and.description
if they are available e.g.Steps to Reproduce (for bugs)
replyWithError
to reproduce e.g.Context
Made it hard to debug some connection issues without adding additional output statements to see the underlying cause of the error. Similarly to #54 where the whole error object has been output because the message
error happened in your connection
is insufficient to identify the issue.Your Environment
The text was updated successfully, but these errors were encountered: