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 an optioin to hide the actual query or query values. #13

Closed
srinathman opened this issue Sep 21, 2012 · 2 comments
Closed

Add an optioin to hide the actual query or query values. #13

srinathman opened this issue Sep 21, 2012 · 2 comments

Comments

@srinathman
Copy link

Sometimes having too much information in the logs can be dangerous and information can somehow be leaked to log files.

One example is when creating a new user, the user password would be written in plain text to the log file:

{"msg-type":"activity","date":"1348244195261","thread-id":"38","query-id":"126","user":"root","priv_user":"root","host":"localhost","ip":"127.0.0.1","cmd":"create_user","query":"CREATE USER 'T'@'%' IDENTIFIED BY 'AAA'"}

{"msg-type":"activity","date":"1348244195262","thread-id":"38","query-id":"127","user":"root","priv_user":"root","host":"localhost","ip":"127.0.0.1","cmd":"grant","query":"GRANT USAGE ON . TO 'T'@'%' IDENTIFIED BY 'AAA' WITH MAX_QUERIES_PER_HOUR 0 MAX_CONNECTIONS_PER_HOUR 0 MAX_UPDATES_PER_HOUR 0 MAX_USER_CONNECTIONS 0"}

But it's still important to audit that the user "T" has been added.

@glicht
Copy link
Contributor

glicht commented Sep 23, 2012

Thanks for logging this issue. I agree that keeping the passwords in a log file is not recommended. I am marking this as an enhancement request.

@glicht
Copy link
Contributor

glicht commented Dec 22, 2014

In version 1.0.6 we added password masking support. There are 2 new configuration options: audit_password_masking_cmds and audit_password_masking_regex to further fine tune this.

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

2 participants