ledgetech / lua-resty-redis-connector

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

Fix DSN parsing in connect function #17

Closed Invizory closed 6 years ago

Invizory commented 6 years ago

Fixes #16.

Invizory commented 6 years ago

I also think it's a good idea to release a new version since the latest v0.06 at luarocks is affected by bugs #14, #16.

hamishforbes commented 6 years ago

Yep you're probably right. Although I'm thinking about moving to OPM only rather than luarocks.

I've also added a luacheck config file and make target that should help catch wrong variable name errors like this.