-
Notifications
You must be signed in to change notification settings - Fork 460
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
readonly mode commands #542
Comments
I attempted modifying
|
Hello, Example output for some commands:
That is why (from this list) "dbsize" only is allowed but not "select" or "flushdb". Maybe the selection criteria should be evaluated to allow info or similar too. But must be make sure to not open up to more dangerous commands... |
thank you , will give it a try. |
Hello, i just added these two commands to a special list set as read-only on dynamic evaluation too. Can you give it a try? And are there more commands missing as read-only? Now the list is defined as a configuration option and can be overwritten with whatever you like inside your |
Hello,
We plan to use info and select commands in readonly mode , I'm not a nodeJs expert, any files that I can modify to make this happen ? currently when READ_ONLY is enabled, the INFO and SELECT commands return this
The text was updated successfully, but these errors were encountered: