patrickpissurno / vredis

Redis client for V, written in V
MIT License
67 stars 10 forks source link

Add auth support #8

Closed drakmail closed 1 year ago

drakmail commented 3 years ago

IDK how to implemet test better in current tests setup, but I'm checked manually authentication (only case without username) and it's worked

gokomer commented 1 year ago

@patrickpissurno We need authentication. Is this PR ok or should we implement another one?

patrickpissurno commented 1 year ago

It got outdated, so we would need a new one

gokomer commented 1 year ago

@drakmail are you willing to work on refactor or should I start work on it?

patrickpissurno commented 1 year ago

By the way, if someone can fix the conflicts this one has and get the build to pass, we could still reuse this one

patrickpissurno commented 1 year ago

Closed in favor of #16