Skip to content

Conversation

@juliusknorr
Copy link
Member

@juliusknorr juliusknorr commented Sep 20, 2022

Make sure that we do not store and use the generated password if it wasn't used for user creation in cases where there is a dbpass already preconfigured

https://github.com/nextcloud/server/blob/bugfix/noid/db-password-install/lib/private/Setup/MySQL.php#L158-L159

@solracsf
Copy link
Member

solracsf commented Sep 20, 2022

Does this fix #34123?

This was referenced Sep 20, 2022
@blizzz blizzz modified the milestones: Nextcloud 25, Nextcloud 26 Sep 22, 2022
@PVince81 PVince81 merged commit 20b7548 into master Sep 28, 2022
@PVince81 PVince81 deleted the bugfix/noid/db-password-install branch September 28, 2022 13:09
@Sephtex
Copy link

Sephtex commented Dec 25, 2022

Does this fix #34123?

I suppose so, in NC24 I have commented out the line

$this->dbPassword = str_shuffle($password);

To bypass the reset of the password, but it seems the fix from this PR would fix it as well.

Maybe if someone can confirm? And maybe this could be backported to NC24 as well so I can avoid patching the source, and @Ma27 who created #34123 as well.

Since #30130, it is causing an issue, as config_is_read_only can't be set to true anymore when running the setup, especially when doing the next order:

  • create a DB user for nextcloud (if not exists)
  • installing nextcloud (DB password gets reset)

@szaimen
Copy link
Contributor

szaimen commented Jan 27, 2023

/backport to stable25

@szaimen
Copy link
Contributor

szaimen commented Jan 27, 2023

/backport to stable24

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants