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

XTR: stats export fails with MySql server gone away #3569

Closed
dasgarner opened this issue Dec 30, 2024 · 0 comments · Fixed by xibosignage/xibo-cms#2905
Closed

XTR: stats export fails with MySql server gone away #3569

dasgarner opened this issue Dec 30, 2024 · 0 comments · Fixed by xibosignage/xibo-cms#2905
Assignees
Labels
Milestone

Comments

@dasgarner
Copy link
Member

After exporting a set of stats, we refresh/reconnect to MySQL and create a media file and upload the exported CSV to the library for storage. When this media file is saved we call audit to create an audit record for that.

Audit logging uses a prepared SQL statement, which is attached to the old MySQL connection and therefore fails.

There are three options:

  1. disable auditing for that operation
  2. add a method to the logging service to clear cached PDO statements and call this before auditing
  3. implement a retry loop in audit logging which does a reconnect

Option number 3 is the most isolated change, although does mean an exception handler.

This also effects v3.

@dasgarner dasgarner added the bug label Dec 30, 2024
@dasgarner dasgarner added this to the 4.2.0-alpha milestone Dec 30, 2024
@dasgarner dasgarner modified the milestones: 4.2.0-alpha, 4.2.0-beta Jan 8, 2025
@dasgarner dasgarner self-assigned this Jan 8, 2025
@maurofmferrao maurofmferrao modified the milestones: 4.2.0-beta, 4.2.0 Jan 15, 2025
@maurofmferrao maurofmferrao modified the milestones: 4.2.0, 4.2.1 Jan 29, 2025
dasgarner added a commit to xibosignage/xibo-cms that referenced this issue Feb 7, 2025
dasgarner added a commit to xibosignage/xibo-cms that referenced this issue Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants