fix: cannot install without db prefix

luceos commented 3 weeks ago

Database prefix and database password can be empty.

Caused by the rewrite in https://github.com/flarum/framework/commit/6f11e044a7c18c14793c7d98e99a83a6ade5ae97, as a consequence some settings of the database config are now incorrectly enforced/required.

Necessity

[ ] Has the problem that is being solved here been clearly explained?
[ ] If applicable, have various options for solving this problem been considered?
[x] For core PRs, does this need to be in core, or could it be in an extension?
[x] Are we willing to maintain this for years / potentially forever?

Confirmed

[ ] Frontend changes: tested on a local Flarum installation.
[ ] Backend changes: tests are green (run composer test).
[ ] Core developer confirmed locally this works as intended.
[ ] Tests have been added, or are not appropriate here.

Required changes:

[ ] Related documentation PR: (Remove if irrelevant)
[ ] Related core extension PRs: (Remove if irrelevant)

luceos commented 3 weeks ago

Allowing no prefix makes sense, are we allowing no password for socket connections?

You can use no password, on development and production. Whether it's advisable has to be seen. Regardless of that decision, a user will receive errors based on their credentials and the credentials the database server expects anyway.. But blatantly demanding a password would potentially block developers with a local dev environment where root has no password.

Ref: MYSQL_ALLOW_EMPTY_PASSWORD on https://hub.docker.com/_/mysql for instance.

SychO9 commented 3 weeks ago

Don't merge yet please. Till the big db ones are merged

flarum / framework

fix: cannot install without db prefix #4001