[#39889] - [4][com_config] validation of the database name
- Closed
- 10 Apr 2024
- Medium
- Build: 4.3-dev
- # 39889
- Diff
- alikon:patch-7
User tests: Successful: Unsuccessful:
Pull Request for Issue #24291 .
Summary of Changes
added check as in #23704
Testing Instructions
com_config->Server change database name with invalid chars like ', /' for mysql and save
for postgresql databasae name must start with a letter, followed by alphanumeric characters
Actual result BEFORE applying this Pull Request
error message "Database connection test failed with the following error: 1049: Could not connect to"
Expected result AFTER applying this Pull Request
warning message "The database name is invalid. It must not contain the following characters: \ / "
Link to documentations
Please select:
-
Documentation link for docs.joomla.org:
-
No documentation changes for docs.joomla.org needed
-
Pull Request link for manual.joomla.org:
-
No documentation changes for manual.joomla.org needed
joomla-cms-bot
- | Category | ⇒ | Administration com_config Language & Strings |
| Status | New | ⇒ | Pending |
| Labels |
Added:
Language Change
PR-4.3-dev
|
||
I have tested this item
Tested successfully in Joomla 4.3.0-beta4-dev of 2 March in MySQL and Postgres databases
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39889.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39889.
| Labels |
Added:
?
?
|
||
| Labels |
Removed:
?
|
||
I have tested this item
Tested in Joomla 4.3.0-beta5-dev of 13 March using PHP 8.1.10
The MySQL site reports / and \ but does not detect . (full stop].
The PostgreSQL site reported invalid format as described in the PR.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39889.
| Labels |
Added:
bug
|
||
This pull request has been automatically rebased to 4.4-dev.
after more than 1 year
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2024-04-10 06:52:01 |
| Closed_By | ⇒ | alikon | |
| Labels |
Added:
Small
PR-4.4-dev
Removed: PR-4.3-dev ? |
||
@alikon I've allowed myself to commit the PHPCS fixes.