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

Add support for RedisInsight Client #159

Open
Dando-Real-ITA opened this issue Nov 16, 2022 · 2 comments
Open

Add support for RedisInsight Client #159

Dando-Real-ITA opened this issue Nov 16, 2022 · 2 comments

Comments

@Dando-Real-ITA
Copy link

Dando-Real-ITA commented Nov 16, 2022

The new Redis Insight GUI client developed by redis is unable to connect to the proxy.
The errors I see in the proxy logs are for missing commands that the client issues at connection, and I tried to pass them using command.conf:

CustomCommand {
  client {
    Mode write
    MinArgs 2
    MaxArgs 6
  }

  module {
    Mode write
    MinArgs 2
    MaxArgs 6
  }

  cluster {
    Mode write
    MinArgs 2
    MaxArgs 6
  }

  sentinel {
    Mode write
    MinArgs 2
    MaxArgs 6
  }

}

But does not seem enough.

@Dando-Real-ITA
Copy link
Author

Update, the new version 2.14 of Redis Insight can load UI, however cannot display the keys because it requires commands memory and dbsize that even if added manually to command.conf, don't work because command.conf does not accept MinArgs 1 commands.

@Dando-Real-ITA
Copy link
Author

I am using a custom fork to support some of the commands directly in Command.cpp

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant