-
Notifications
You must be signed in to change notification settings - Fork 6
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
Detect passwordless Accounts #84
Comments
MySQL 5.5
http://doc.docs.sk/mysql-refman-5.5/grant-table-structure.html MySQL 5.6
https://dev.mysql.com/doc/refman/5.6/en/grant-tables.html MySQL 5.7https://dev.mysql.com/doc/refman/5.7/en/grant-tables.html The MySQL 8.0https://dev.mysql.com/doc/refman/8.0/en/grant-tables.html In 8.0, the password storage default changes to Authentication got really compliated in 8.0:
|
Accounts without passwords should be flagged to avoid unintended exposure. This goes doubly for accounts that are accessible outside of localhost. We don't currently case too much about what they have access to. We could examine if they only have read access (and whether it's limited to "monitoring" schemas like mysql or performance_schema) but that's a future refinement once we see what hits we get. Also required some refactoring of various bits of code and some hint at the newer structure for data objects (heading towards using readonly classes, using proper enums and named parameters which means this also bumps our requirements up to PHP 8.* as well). Ideally I want to be able to support multiple data sources (database, SQL definition files, etc) without massive refactoring.
Tasks:
|
Added Passwordless Account check (#84)
Cadfael should detect accounts with no password.
In MySQL 8.0
authentication_string
field.In MySQL 5.7
authentication_string
field.In MySQL 5.6
Password
field, if theplugin
field has the valuemysql_native_password
.In MySQL 5.5
Password
field, if theplugin
field isNULL
or empty.The text was updated successfully, but these errors were encountered: