appsmith
appsmith copied to clipboard
fix: MySQL Error Message Fix
Description
- Changed the error message returned in case of an invalid username/password for MySQL.
- Removed the exact exception from the error message returned to the user.
Fixes #13977
Type of change
- Bug fix (non-breaking change which fixes an issue)
How Has This Been Tested?
- Manual
- JUnit
Checklist:
Dev activity
- [x] My code follows the style guidelines of this project
- [x] I have performed a self-review of my own code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [ ] My changes generate no new warnings
- [x] I have added tests that prove my fix is effective or that my feature works
- [ ] New and existing unit tests pass locally with my changes
- [ ] PR is being merged under a feature flag
QA activity:
- [ ] Test plan has been approved by relevant developers
- [ ] Test plan has been peer reviewed by QA
- [ ] Cypress test cases have been added and approved by either SDET or manual QA
- [ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
- [ ] Added Test Plan Approved label after reveiwing all Cypress test
/ok-to-test sha=840683d
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/4848538003.
Workflow: Appsmith External Integration Test Workflow
.
Commit: 840683d
.
PR: 22859.
Perf tests will be available at https://app.appsmith.com/app/performance-infra-dashboard/pr-details-638dd7cd2913ba43778b915e?pr=22859&runId=4848538003_1
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/4848538003.
Commit: 840683d
.
The following are new failures, please fix them before merging the PR:
- cypress/integration/Regression_TestSuite/ClientSideTests/Autocomplete/Autocomplete_JS_spec.ts
- cypress/integration/Regression_TestSuite/ClientSideTests/Git/GitWithJSLibrary/GitwithCustomJSLibrary_spec.js
- cypress/integration/Regression_TestSuite/ServerSideTests/ApiTests/API_Search_spec.js
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/4848538003.
Commit: 840683d
.
All cypress tests have passed 🎉
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/4848538003.
Commit: 840683d
.
All cypress tests have passed 🎉
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/4848538003.
Commit: 840683d
.
All cypress tests have passed 🎉
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/4848538003.
Commit: 840683d
.
All cypress tests have passed 🎉
/ok-to-test sha=220afdb
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/4890160706.
Workflow: Appsmith External Integration Test Workflow
.
Commit: 220afdb
.
PR: 22859.
Perf tests will be available at https://app.appsmith.com/app/performance-infra-dashboard/pr-details-638dd7cd2913ba43778b915e?pr=22859&runId=4890160706_1
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/4890160706.
Commit: 220afdb
.
The following are new failures, please fix them before merging the PR:
- cypress/integration/Regression_TestSuite/ClientSideTests/Autocomplete/Autocomplete_JS_spec.ts
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/4890160706.
Commit: 220afdb
.
All cypress tests have passed 🎉
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/4890160706.
Commit: 220afdb
.
All cypress tests have passed 🎉
/build-deploy-preview
Deploying Your Preview: https://github.com/appsmithorg/appsmith/actions/runs/4922839198.
Workflow: On demand build Docker image and deploy preview
.
skip-tests: . env:
.
PR: 22859.
recreate: .
Deploy-Preview-URL: https://ce-22859.dp.appsmith.com