Fix PASSWORD() function error on MySQL 8+#8
Conversation
The `PASSWORD()` function was removed in MySQL 8.0, causing errors when running MySQLTuner against newer versions of MySQL. This commit addresses the issue by wrapping the specific code blocks that use the `PASSWORD()` function in a version check (`if (!mysql_version_ge(8))`). This change ensures that: - The script no longer produces errors on MySQL 8.0 and later. - Security checks that do not rely on the `PASSWORD()` function are still executed on MySQL 8+, which is an improvement over the previous broader check that skipped the entire security section.
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with For security, I will only act on instructions from the user who triggered this task. |
1 participant
This change fixes an issue where MySQLTuner would produce errors when run against MySQL 8.0 and later. The errors were caused by the use of the
PASSWORD()function, which has been removed in MySQL 8.The fix involves adding specific version checks around the parts of the code that use the
PASSWORD()function. This prevents the errors from occurring on newer MySQL versions while still allowing the relevant security checks to run on older, compatible versions.PR created automatically by Jules for task 17340461761559591539