chore: use netty.version property instead of hardcoded version in mysqlPlugin

[khushalmajoka]

Fixes #41901

Problem

The mysqlPlugin module hardcoded an outdated Netty version (4.1.75.Final) for the netty-resolver-dns-native-macos dependency, bypassing the centrally managed ${netty.version} property in the parent pom.xml. This outdated version is flagged with known CVEs.

Fix

Replaced the two hardcoded 4.1.75.Final version strings with ${netty.version}, which is already pinned to 4.1.135.Final in the parent pom.xml specifically to address these CVEs. This keeps the dependency in sync with the rest of the project going forward.

Testing

Ran mvn -pl appsmith-plugins/mysqlPlugin -am install -DskipTests — build succeeds with no compilation or dependency resolution issues.

Summary by CodeRabbit

• Chores
  • Improved consistency for macOS-specific networking components by aligning them with the application’s centrally managed versioning.
  • This helps maintain reliable builds and reduces the risk of compatibility issues across supported macOS environments.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: fd9eae83-787e-4e06-a323-26a3b132cb75

📥 Commits

Reviewing files that changed from the base of the PR and between d2ec04c and 98c6e54.

📒 Files selected for processing (1)

• app/server/appsmith-plugins/mysqlPlugin/pom.xml

---

Walkthrough

The MySQL plugin’s two macOS native Netty DNS dependencies now use the ${netty.version} Maven property instead of the fixed 4.1.75.Final version.

Changes

Netty dependency alignment

Layer / File(s)	Summary
Align macOS Netty dependency versions app/server/appsmith-plugins/mysqlPlugin/pom.xml	Both osx-x86_64 and osx-aarch_64 Netty DNS native dependencies now reference ${netty.version}.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

• appsmithorg/appsmith#41928: Centralizes the MySQL plugin’s Netty version usage.
• appsmithorg/appsmith#41947: Updates Netty dependency declarations in Maven configuration.

Suggested labels: ok-to-test

Poem

Two native paths now share one tune,
Netty versions align beneath the moon.
x86 and ARM agree,
A tidier POM for all to see.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Linked Issues check	⚠️ Warning	The change fixes mysqlPlugin's hardcoded Netty version, but the linked issue also calls out awsLambdaPlugin HTTP codec CVEs.	Also update the awsLambdaPlugin Netty HTTP codec dependency, or narrow the issue scope if mysqlPlugin was the only intended fix.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly describes the Netty version cleanup in mysqlPlugin.
Description check	✅ Passed	The description covers problem, fix, testing, and the issue reference, so it is mostly complete.
Out of Scope Changes check	✅ Passed	The only code change is the Netty version substitution in mysqlPlugin, which matches the stated objective.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[subrata71]

/build-deploy-preview skip-tests=true

[github-actions]

Deploying Your Preview: https://github.com/appsmithorg/appsmith/actions/runs/29437658006.
Workflow: On demand build Docker image and deploy preview.
skip-tests: true.
env: ``.
PR: 41999.
recreate: .
base-image-tag: .

[github-actions]

Deploy-Preview-URL: https://ce-41999.dp.appsmith.com