search

moaxaca / async-redis

First class async & promise support for redis.
Other
168 stars 21 forks source link

Connecting to a non-localhost Redis instance #2

Closed fishnux closed 6 years ago

fishnux commented 6 years ago 
const asyncRedisClient = require("async-redis").createClient(1111, '8.8.8.8');

It looks like the second argument ("8.8.8.8") is being ignored:

Error: Redis connection to 127.0.0.1:1111 failed - connect ECONNREFUSED 127.0.0.1:1111

Edit: Just found this workaround:

const asyncRedisClient = require("async-redis").createClient(`redis://8.8.8.8:1111`);
moaxaca commented 6 years ago

@fishnux I will work this ASAP ty for notice.

moaxaca commented 6 years ago

Fixed with https://github.com/moaxaca/async-redis/commit/8a645340325b58663831f6153f26c301e03575c8 Release with https://github.com/moaxaca/async-redis/commit/3dc51f429aaa081cb4cf68b884b5ce4b847c579e

Closing Issue