-
-
Notifications
You must be signed in to change notification settings - Fork 15.4k
nixos/mysql: support package=mysql57 #49735
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
Conversation
There is a problem with this solution: it won't work for MySQL versions that are older than the current |
dd3c038
to
9897efe
Compare
The problem mentioned above is now corrected. I don't like specifying |
Passing |
@lheckemann ok |
9897efe
to
21e9c35
Compare
@lheckemann I added an explanation about |
Thanks! Does this mean that any local user will be able to log in as the root db user without a password then? |
Yes. This is how the default configuration (with |
Hmm well upstream's change seems like a sensible one then. Maybe we should do some cc @thoughtpolice mariadb maintainer |
To me, it looks like an over-cautious default option whose primary purpose is to make the admin pay attention by giving him a deployment that is not instantly usable. The admin is then forced to either set the password explicitly, or explicitly request empty password. Real-world installation will have a better mechanism for setting up the password. For example, Gentoo interactively asks for password; Ubuntu uses socket-based authentication; MySQL's official APT package also interactively asks for password, and switches to socket-based authentication if no password is provided. (On a separate note, what should we consider "upstream" - Oracle' MySQL or MariaDB?..)
I would be quite upset if this logic would just call On the other hand, it makes sense to make use of socket-based authentication, which is a convenient mechanism for granting automatic and secure access to the system's In any case, this may become a topic for another PR; for this one, initializing with empty password is the most logical thing to do. |
I was thinking less along the lines of randomized password and more along the lines of an |
This also fixes #25418. Will backport to 18.09. |
Resolves #49732.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)