Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Authentication issue with Predixy server #358

Open
l2dy opened this issue Dec 15, 2022 · 0 comments
Open

Authentication issue with Predixy server #358

l2dy opened this issue Dec 15, 2022 · 0 comments
Labels

Comments

@l2dy
Copy link

l2dy commented Dec 15, 2022

Version

4.3.5

Context

I encountered an authentication issue when connecting to a Predixy-proxied Redis cluster.

Do you have a reproducer?

No, but could provide one if requested.

Steps to reproduce

  1. Configure a predixy cluster with authentication enabled.
  2. Connect to the proxy with a Redis URI containing the password.
  3. Issue a PING command via Vert.x Redis client.
  4. Get ERR auth permission deny error.

Extra

if (err instanceof ErrorType) {
if (((ErrorType) err).is("NOAUTH")) {
// old authentication required

The Redis connection manager in Vert.x Redis client expects the server to return NOAUTH for an unauthenticated client if HELLO is not recognized by the server. Predixy, however, returns an error of type ERR, so old-style authentication is not attempted at all in the ping() method.

Is it possible to attempt old-style authentication before attempting to PING the server if password is set?

Full error message from Predixy is ERR auth permission deny, as shown in https://github.com/joyieldInc/predixy/blob/c15f54e274afc596b8e8cd7fecac70164c95b9ec/src/Response.cpp#L27.

@l2dy l2dy added the bug label Dec 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

1 participant