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

Bug 1435099: Audit Log events do not report default DB #649

Merged
merged 1 commit into from Jul 22, 2016

Conversation

gl-sergei
Copy link
Contributor

Blueprint
https://blueprints.launchpad.net/percona-server/+spec/audit-log-default-db

Add field DB for MYSQL_AUDIT_GENERAL_STATUS records.

@gl-sergei
Copy link
Contributor Author

@laurynas-biveinis laurynas-biveinis self-assigned this Jul 11, 2016
connection default;

--exec $MYSQL --user=user1 --password=111 test -e "use db1; SELECT * FROM t;"
# is there better way to check that server reset connection initiated by mysql cli?
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wrapping exec with include/count_connections and include/wait_until_connection_count should do this

@laurynas-biveinis
Copy link
Contributor

No testcase for the install plugin with an open parallel connection?

@gl-sergei gl-sergei force-pushed the 5.6-ps-audit-log-default-db branch 4 times, most recently from a92aa36 to 807084d Compare July 13, 2016 10:19
@gl-sergei
Copy link
Contributor Author

  • fixed typo
  • changed test to use utf8
  • added test for database 'ąžąžąžą'
  • surrounded --exec mysql with --source include/count_sessions.inc/--source include/wait_until_count_sessions.inc
  • added test for INSTALL PLUGIN

jenkins build: http://jenkins.percona.com/view/PS%205.6/job/percona-server-5.6-param/1231/

GRANT ALL PRIVILEGES ON ąžąžąžą.* TO 'user2'@'%';

# truncate audit log
UNINSTALL PLUGIN audit_log;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This gives "plugin is busy and will be uninstalled on shutdown"

@laurynas-biveinis
Copy link
Contributor

Something else about encodings, in e.g. XML the header is . If the server encoding is different, are the strings transcoded before logging?

@gl-sergei
Copy link
Contributor Author

Something else about encodings, in e.g. XML the header is . If the server encoding is different, are the strings transcoded before logging?

See my comment above:

So, there is another bug! Query is written to log in client charset while other fields in utf8.

@laurynas-biveinis
Copy link
Contributor

Ah, OK, it's the same thing we are talking about :) Have you logged the bug?

@gl-sergei gl-sergei force-pushed the 5.6-ps-audit-log-default-db branch from 807084d to 9f9e2c2 Compare July 14, 2016 08:41
@gl-sergei
Copy link
Contributor Author

@laurynas-biveinis
Copy link
Contributor

LGTM

@gl-sergei gl-sergei merged commit 1b895ee into percona:5.6 Jul 22, 2016
@gl-sergei gl-sergei deleted the 5.6-ps-audit-log-default-db branch November 24, 2016 01:59
inikep pushed a commit to inikep/percona-server that referenced this pull request Apr 23, 2020
Summary:
Changes:
* reset m_detailed_error to empty string
* added tests to verify snapshot conflicts message disappears on consecutive deadlocks
Closes facebook/mysql-5.6#649

Differential Revision: D5383202

fbshipit-source-id: aff56938297
inikep pushed a commit to inikep/percona-server that referenced this pull request Feb 24, 2021
Summary:
Changes:
* reset m_detailed_error to empty string
* added tests to verify snapshot conflicts message disappears on consecutive deadlocks
Closes facebook/mysql-5.6#649

Differential Revision: D5383202

fbshipit-source-id: aff56938297
inikep pushed a commit to inikep/percona-server that referenced this pull request Nov 15, 2021
…#649)

Summary:
Changes:
* reset m_detailed_error to empty string
* added tests to verify snapshot conflicts message disappears on consecutive deadlocks
Closes facebook/mysql-5.6#649

Differential Revision: D5383202 (facebook/mysql-5.6@1328f8f)

fbshipit-source-id: a4868703e93
ldonoso pushed a commit to ldonoso/percona-server that referenced this pull request Mar 15, 2022
…#649)

Summary:
Changes:
* reset m_detailed_error to empty string
* added tests to verify snapshot conflicts message disappears on consecutive deadlocks
Closes facebook/mysql-5.6#649

Differential Revision: D5383202
inikep pushed a commit to inikep/percona-server that referenced this pull request Apr 9, 2024
…#649)

Summary:
Changes:
* reset m_detailed_error to empty string
* added tests to verify snapshot conflicts message disappears on consecutive deadlocks
Closes facebook/mysql-5.6#649

Differential Revision: D5383202
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants