MySQL security enhancement

[xiangyan99]

What does this PR do?

[Provide a clear, concise description of the changes]

[Any additional context, screenshots, or information that helps reviewers]

1. Disallow comments
2. Disallow keywords: UNION/INTERSECT/EXCEPT

GitHub issue number?

[Link to the GitHub issue this PR addresses]

Pre-merge Checklist

• Required for All PRs
  • Read contribution guidelines
  • PR title clearly describes the change
  • Commit history is clean with descriptive messages (cleanup guide)
  • Added comprehensive tests for new/modified functionality
  • Updated servers/Azure.Mcp.Server/CHANGELOG.md and/or servers/Fabric.Mcp.Server/CHANGELOG.md for product changes (features, bug fixes, UI/UX, updated dependencies)
• For MCP tool changes:
  • One tool per PR: This PR adds or modifies only one MCP tool for faster review cycles
  • Updated servers/Azure.Mcp.Server/README.md and/or servers/Fabric.Mcp.Server/README.md documentation
  • Validate README.md changes using script at eng/scripts/Process-PackageReadMe.ps1. See Package README
  • Updated command list in /servers/Azure.Mcp.Server/docs/azmcp-commands.md and/or /docs/fabric-commands.md
  • Run .\eng\scripts\Update-AzCommandsMetadata.ps1 to update tool metadata in azmcp-commands.md (required for CI)
  • For new or modified tool descriptions, ran ToolDescriptionEvaluator and obtained a score of 0.4 or more and a top 3 ranking for all related test prompts
  • For tools with new names, including new tools or renamed tools, update consolidated-tools.json
  • For renamed tools, follow the Tool Rename Checklist and tag the PR with the breaking-change label
  • For new tools associated with Azure services or publicly available tools/APIs/products, add URL to documentation in the PR description
• Extra steps for Azure MCP Server tool changes:
  • Updated test prompts in /servers/Azure.Mcp.Server/docs/e2eTestPrompts.md
  • 👉 For Community (non-Microsoft team member) PRs:
    • Security review: Reviewed code for security vulnerabilities, malicious code, or suspicious activities before running tests (crypto mining, spam, data exfiltration, etc.)
    • Manual tests run: added comment /azp run mcp - pullrequest - live to run Live Test Pipeline

[Copilot]

Pull request overview

Enhances the MySQL query safety validator to reduce SQL-injection bypass vectors (notably comment-based obfuscation and set-operator exfiltration patterns) in the Azure MCP MySQL tool.

Changes:

• Block SQL comments (--, #, /* ... */, and MySQL version comments /*!...*/) rather than stripping them.
• Add set-operator keywords (UNION/INTERSECT/EXCEPT) to the MySQL dangerous keyword blocklist and switch keyword/function detection to regex-based word-boundary matching.
• Expand unit tests to cover string-literal edge cases and add explicit tests for comment and set-operation rejection; update Azure MCP Server changelog entry.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File	Description
tools/Azure.Mcp.Tools.MySql/src/Services/MySqlService.cs	Updates query validation logic to reject comments and detect blocked keywords/functions via regex patterns.
tools/Azure.Mcp.Tools.MySql/tests/Azure.Mcp.Tools.MySql.UnitTests/Services/MySqlServiceQueryValidationTests.cs	Adds/adjusts tests for comment rejection, set operations, and reduced false positives in string literals.
servers/Azure.Mcp.Server/CHANGELOG.md	Documents the MySQL query validation security fix in the Unreleased changelog.