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

Support renaming commands #16

Closed
logdnalf opened this issue Apr 27, 2020 · 2 comments
Closed

Support renaming commands #16

logdnalf opened this issue Apr 27, 2020 · 2 comments

Comments

@logdnalf
Copy link

Renaming commands is not supported and specifically ignored by the Redis Operator:

Can we remove this restriction? Renaming commands is quite useful and can improve security.

@nrvnrvn
Copy link
Collaborator

nrvnrvn commented May 25, 2020

Hi! I think it is possible. Initially I introduced this restriction to protect some essential configuration options. Excluding the rename-command was a way to protect a user from shooting their leg. On the other hand we have two options for removing this restriction:

  1. allow rename-command as is
  2. allow rename-command but protect some essential commands like REPLICAOF.

The first option is super easy and it will be sole user's responsibility to apply any renamings with caution.
The second option implies introducing some parsing logic and a list of excluded commands but I am not sure if it is worth spending time on it.

@nrvnrvn
Copy link
Collaborator

nrvnrvn commented Jun 15, 2022

superseded by nrvnrvn/k8dis#3

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

2 participants