Skip to content
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

Recover installation when creating the database user fails and improve password strength #33513

Conversation

nickvergessen
Copy link
Member

I reinstalled my setup and ended up with this error on MySQL:

General error: 1819 Your password does not satisfy the current policy requirements

The problem is the exception was logged and swallowed, so the remaining part of the installation process tried to grant privileges to that user and wrote it to the config files resulting afterwards in:

MySQL username and/or password not valid
 -> You need to enter details of an existing account.
Trace: #0 /home/nickv/Nextcloud/25/server/lib/private/Setup.php(353): OC\Setup\MySQL->setupDatabase()
#1 /home/nickv/Nextcloud/25/server/core/Command/Maintenance/Install.php(105): OC\Setup->install()
#2 /home/nickv/Nextcloud/25/server/3rdparty/symfony/console/Command/Command.php(255): OC\Core\Command\Maintenance\Install->execute()
#3 /home/nickv/Nextcloud/25/server/3rdparty/symfony/console/Application.php(1009): Symfony\Component\Console\Command\Command->run()
#4 /home/nickv/Nextcloud/25/server/3rdparty/symfony/console/Application.php(273): Symfony\Component\Console\Application->doRunCommand()
#5 /home/nickv/Nextcloud/25/server/3rdparty/symfony/console/Application.php(149): Symfony\Component\Console\Application->doRun()
#6 /home/nickv/Nextcloud/25/server/lib/private/Console/Application.php(213): Symfony\Component\Console\Application->run()
#7 /home/nickv/Nextcloud/25/server/console.php(100): OC\Console\Application->run()
#8 /home/nickv/Nextcloud/25/server/occ(11): require_once('...')
#9 {main}

As per https://ostechnix.com/fix-mysql-error-1819-hy000-your-password-does-not-satisfy-the-current-policy-requirements/ there are 3 levels of password strength and the new default is medium:

MEDIUM Length >= 8, numeric, mixed case, and special characters.

So in a second commit instead of reusing the provided root account, I increased the random password strength that hopefully also matches the STRONG rules

Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen added this to the Nextcloud 25 milestone Aug 12, 2022
@nickvergessen nickvergessen requested review from PVince81, a team, skjnldsv and come-nc and removed request for a team August 12, 2022 08:05
@nickvergessen
Copy link
Member Author

/backport to stable24

@nickvergessen
Copy link
Member Author

/backport to stable23

Copy link
Member

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@PVince81 PVince81 added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Aug 12, 2022
@nickvergessen nickvergessen merged commit 478690b into master Aug 12, 2022
@nickvergessen nickvergessen deleted the bugfix/noid/recover-installation-when-creating-the-user-failed branch August 12, 2022 09:02
@skjnldsv skjnldsv mentioned this pull request Aug 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish bug feature: install and update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants