-
Notifications
You must be signed in to change notification settings - Fork 25
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
Add MariaDB socket #30
base: master
Are you sure you want to change the base?
Conversation
Make socket the default connection method, and keep user/pass as an option.
This would break current deployments. If you want to do this, better to have it as a configurable option and leave the defaults unchanged (so existing deployments are not affected). This needs to be done in a backwards compatible way. |
Hmm, but existing deployments should already have a config file? This only affects new installations, or am I missing something here? |
Sorry, I misread the commit and thought it was changing the code. Dangers of doing this on a mobile phone… But now I am wondering why we are changing the default provided in the config? Perhaps better to just provide some documentation (comments) there to give some examples for different environments? |
I was thinking since SOCKET is recommended due to security, maybe that's what we (you) want to advocate? I intend to contribute to this repo and improve it were I can, just need to get my head around perl. I'm more of a sysadmin guy. :) |
In the long run, maybe we can provide an easy way to setup the server, like a preconfigured VM or something. I've done it in the past with Nextcloud since I love that project. And since I'm now invested in this as well I have much to gain on getting a wider spread so that more people can use it - and contribute. Then on the other hand, maybe it's not a good idea to invest in the V2 server at all? |
I think we should keep the TCP/IP scheme as the default, because it's the default MySQL & MariaDB setup, and most distributions & sysadmins keep this default, so Also, a TCP loopback connection to |
I hear you. Made some changes, please have a look. 👍 Please squash and commit. Got wild with the commits here. :) |
Make socket the default connection method, and keep user/pass as an option.
mysql_secure_installation
recommends using a socket instead.