-
-
Notifications
You must be signed in to change notification settings - Fork 123
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
Enable logging #150
Enable logging #150
Conversation
recipes/_audit_plugin.rb
Outdated
@@ -14,20 +14,25 @@ | |||
audit_plugin_options['server_audit_syslog_priority'] = \ | |||
node['mariadb']['audit_plugin']['server_audit_syslog_priority'] | |||
|
|||
audit_plugin_options['enable'] = '#server_audit_logging = ON' | |||
audit_plugin_options['server_audit_logging'] = 'ON' | |||
|
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
recipes/_audit_plugin.rb
Outdated
@@ -44,6 +49,7 @@ | |||
node['mariadb']['audit_plugin']['server_audit_syslog_facility'] + '\';' \ | |||
'SET GLOBAL server_audit_syslog_priority=\'' + \ | |||
node['mariadb']['audit_plugin']['server_audit_syslog_priority'] + '\';"' \ | |||
'SET GLOBAL server_audit_logging=\'ON\';' \ |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
The tests need some fixing before this can be merged. Some
|
Included in v1.3.0 release on supermarket. Thanks! |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
This enables logging by default in this recipe, not sure why it wasn't by default, I mean why bother configuring all this logging if you don't want to use it? Also adds the password and user to the mysql commands and marks the execute as sensitive.