search

ledgetech / lua-resty-redis-connector

Connection utilities for lua-resty-redis
234 stars 71 forks source link

fix auth bug #2

Closed vkill closed 9 years ago

vkill commented 9 years ago

issue#1

pintsized commented 9 years ago

Thanks for this. I implemented it slightly differently, choosing not to allow the password to appear in the logs, but also to return the connected instance + err, so that the user is responsible for closing the connection, or re-attempting auth. That is, a failed "auth" command doesn't mean the connection has been closed.

https://github.com/pintsized/lua-resty-redis-connector/commit/241de600b93a0cf0ed4c041bb2125baf55261bf1