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

Readonly Admin #463

Closed
renecannao opened this issue Jan 9, 2016 · 1 comment
Closed

Readonly Admin #463

renecannao opened this issue Jan 9, 2016 · 1 comment

Comments

@renecannao
Copy link
Contributor

Admin interface has 2 ways of operate:

  • if using admin credentials, it is in read/write mode
  • if using stats credentials, it is in read only mode

As a requirement to have a ProxySQL Cluster , admin module could be in read only mode even using admin credentials

@renecannao
Copy link
Contributor Author

Features added:

  • added new global variable admin-read_only: it is possible to enable read_only changing this variable and running LOAD ADMIN VARIABLES TO RUNTIME
  • admin interface now replies to SHOW GLOBAL VARIABLES LIKE 'read_only'
  • added new command PROXYSQL READONLY to enable read only mode
  • added new command PROXYSQL READWRITE to disable read only mode : this is the only way to disable read only mode while global_variables is not readable

Note that this doesn't prevent the execution of commands LOAD and SAVE : these commands need to be used with caution

renecannao added a commit that referenced this issue Jan 10, 2016
Features added:
* added new global variable admin-read_only: it is possible to enable read_only changing this variable and running LOAD ADMIN VARIABLES TO RUNTIME
* admin interface now replies to SHOW GLOBAL VARIABLES LIKE 'read_only'
* added new command PROXYSQL READONLY to enable read only mode
* added new command PROXYSQL READWRITE to disable read only mode : this is the only way to disable read only mode while global_variables is not readable

Note that this doesn't prevent the execution of commands LOAD and SAVE : these commands need to be used with caution
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant