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

Added ubuntu 22.04 support & Fix mysql connection problem && Fix permission problem #99

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

rayfalling
Copy link

No description provided.

@rayfalling
Copy link
Author

Another problem: why images in docker hub is about 3 years ago?

@@ -405,13 +410,17 @@ else

mysql -u root -e "CREATE USER 'zmuser'@'localhost' IDENTIFIED BY 'zmpass';"
mysql -u root -e "GRANT ALL PRIVILEGES ON *.* TO 'zmuser'@'localhost';"
mysql -u root -e "ALTER USER 'zmuser'@'localhost' IDENTIFIED WITH mysql_native_password BY 'zmpass';"
Copy link
Member

Choose a reason for hiding this comment

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

Why? mysql_native_password is deprecated in modern mysql.

Copy link
Author

Choose a reason for hiding this comment

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

image

Because if the use of mysql_native_password is not allowed, Mysql will fail to initialize the connection.

Copy link
Author

Choose a reason for hiding this comment

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

Maybe you should update database connection code in zoneminder?

Choose a reason for hiding this comment

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

https://dev.mysql.com/doc/refman/8.0/en/caching-sha2-pluggable-authentication.html seems like the now-default auth mechanism requires either a TLS connection or RSA-based auth, neither of which are worth it at all for a connection to a DB server inside the same container. It makes sense to switch it back to mysql_native_password imo to keep things simple.

Copy link
Member

Choose a reason for hiding this comment

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

mysql_native_password is DEPRECATED. If you use it with modern mysql your logs will be spammed endlessly about it.

@connortechnology
Copy link
Member

Docker hub started charging and we can't be bothered to figure out how to use them going forward. Better to use something else.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants