-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[5.3] Installation - Admin password must be at least 12 characters long #46173
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: 5.3-dev
Are you sure you want to change the base?
Conversation
@alikon Please do not use hard-coded strings. This should be a variable so that appropriate translations can be made in all languages. |
@tecpromotion no more hard-coded strings |
I have tested this item ✅ successfully on 65a2286 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46173. |
Does it make sense to check for invalid characters here as well, or do we need a new PR for that? |
the only chars that is been checked on installation should be spaces, check added |
It works in principle, but the string |
then the text of as it is only used here joomla-cms/libraries/src/Form/Rule/PasswordRule.php Lines 95 to 115 in 1b14ada
it should be |
Pull Request for Issue #46111, #42289 .
Summary of Changes
Admin password must be at least 12 characters long and not contains spaces
Testing Instructions
Install Joomla 5.3.x
At Login Data: set a short <12 char and or use spaces as 'Set the username for your Super User account. *'
Actual result BEFORE applying this Pull Request
The installation process goes on until it gets stuck in the 'Installation progress' phase. There is no possibility to cancel, stop or do something other than go back in the browser.
Expected result AFTER applying this Pull Request
you cannot proceed untill you set a minimum password of 12 chars and without spaces

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