andyet/thoonk.js — pull request #14 by mrtidy (2011-12-29 10:04:07+00:00)
Title: add optional 4th param for Redis auth support

$ grep ^requirepass /etc/redis/redis.conf
requirepass booman

For this case, if I only provide Thoonk with hostname and port, then I get an error from RedisClient:

> Error Ready check failed: Error: ERR operation not permitted

I verified that existing tests all still pass as well as possible but I haven't committed the modifications of the tests supporting passwords because of a node_redis issue ( https://github.com/mranney/node_redis/issues/155 ): test_jobs uses 'ready' and current versions of node_redis fail due to an auth issue.

Thank you.

- jeremiah