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

MySQL improvements #65

Merged
merged 7 commits into from
May 26, 2022
Merged

MySQL improvements #65

merged 7 commits into from
May 26, 2022

Conversation

github-actions[bot]
Copy link

@github-actions github-actions bot commented May 25, 2022

This PR Closes #64 and implements improvements MySQL authentication. There are more tools than not out there that gets confused with passwordless databases, with PR we are adding a password for remote authentication, while authentication using root user on localhost remains passwordless.

1. root@localhost

Localhost authentication for root user is passwordless, using auth_socket authentication plugin, for more info see: https://dev.mysql.com/doc/refman/8.0/en/socket-pluggable-authentication.html

2. admin@% and admin@localhost

A new admin user has been added to handle remote and none privileged authentication on localhost, using the mysql_native_passwordplugin, for more info see: https://dev.mysql.com/doc/refman/8.0/en/native-pluggable-authentication.html

See: https://medium.com/@crmcmullen/how-to-run-mysql-8-0-with-native-password-authentication-502de5bac661

Credentials

  • user: admin
  • pass: admin

Can be configured using the MYSQL_USER and MYSQL_PASS env variable.

mbind: Operation not permitted messages in logs

Issue and the solution is explained here: https://stackoverflow.com/questions/55559386/how-to-fix-mbind-operation-not-permitted-in-mysql-error-log

From the thread:

Without CAP_SYS_NICE enabled, MySQL won't be able to handle thread priorities, and I guess you will then loose a new feature introduced with MySQL 8 (I believe Resource Groups dev.mysql.com/doc/refman/8.0/en/resource-groups.html). MySQL should warn us once during startup. Instead (not sure whether it is still the case nowadays), it fills the disk with this message.

@github-actions github-actions bot added the refactor Needs to be improved label May 25, 2022
@ercanozkaya ercanozkaya requested a review from johanjanssens May 25, 2022 16:58
Copy link
Member

@johanjanssens johanjanssens left a comment

Choose a reason for hiding this comment

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

One small change rest is looking great

config/s6/cont-init.d/102-app.sh Outdated Show resolved Hide resolved
@ercanozkaya ercanozkaya merged commit c1dcfc3 into master May 26, 2022
@ercanozkaya ercanozkaya deleted the feature/64-mysql branch May 26, 2022 11:53
johanjanssens added a commit that referenced this pull request May 26, 2022
johanjanssens added a commit that referenced this pull request May 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Needs to be improved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MySQL improvements
2 participants