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

4.0.0-3 broken single connection with ECONNREFUSED #668

Closed
Delagen opened this issue Jul 25, 2018 · 4 comments
Closed

4.0.0-3 broken single connection with ECONNREFUSED #668

Delagen opened this issue Jul 25, 2018 · 4 comments

Comments

@Delagen
Copy link

Delagen commented Jul 25, 2018

connect to single node at 127.0.0.1 with auth.
version 4.0.0-2 works as expected.

@luin
Copy link
Collaborator

luin commented Aug 1, 2018

Hi, could you enable the logs (DEBUG=ioredis:* node yourapp.js) and post the logs here?

@Delagen
Copy link
Author

Delagen commented Aug 1, 2018

[DEBUG] status[redis.domain.ru:6379]: connecting -> close +9ms
[DEBUG] reconnect in 36ms +2ms
[DEBUG] status[redis.domain.ru:6379]: close -> reconnecting +1ms
[DEBUG] error: Error: connect EADDRNOTAVAIL 127.0.0.1 - Local (0.0
.0.0:55277) +5ms
[ERROR] Error: connect EADDRNOTAVAIL 127.0.0.1

  • Local (0.0.0.0:55277)
    at internalConnect (net.js:932:16)
    at defaultTriggerAsyncIdScope (internal/async_hooks.js:294:19)
    at GetAddrInfoReqWrap.emitLookup [as callback] (net.js:1081:9)
    at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:61:10)

@Delagen
Copy link
Author

Delagen commented Aug 1, 2018

This is due

function isIIpcConnectionOptions(value) {
    return value.hasOwnProperty('path');
}

because I supply property path with null value

Fixed with remove null|undefined options before connect.
So it may be closed
But manual look like

Local domain socket path. If set the port, host and family will be ignored.

@luin luin closed this as completed in d40a99e Aug 2, 2018
@luin
Copy link
Collaborator

luin commented Aug 2, 2018

Fixed. Will be releaseed in the next version. Thanks for pointing this out!

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

No branches or pull requests

2 participants