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

Report if files (core, theme, plugins) don't match checksums #382

Open
paulschreiber opened this issue May 25, 2020 · 3 comments
Open

Report if files (core, theme, plugins) don't match checksums #382

paulschreiber opened this issue May 25, 2020 · 3 comments

Comments

@paulschreiber
Copy link
Contributor

Feature request/bug description

Similar to WP-CLI's verify-checksums feature
https://developer.wordpress.org/cli/commands/core/verify-checksums/

It would be helpful if Health Check warned you when a checksum mismatch occurred. A bunch of sites I manage had wp-includes/general-template.php altered, and Site Health didn't warn of the problem.

@Zodiac1978
Copy link
Contributor

This is part of the Health Check plugin since #41 - but only for core WordPress files.

Themes and plugins are possible but they seem to be creating much more false positives:
#41 (comment)
Therefore the /wp-content folder is not checked at all, I think.

And this feature needs to be manually started. There is no schedule at the moment. If I remember correctly this was classified as plugin territory. Maybe this feature request could be revisited again.

@paulschreiber
Copy link
Contributor Author

@Zodiac1978 from a security POV, this is very important. It'd be good to check core filess at least hourly.

@Clorith
Copy link
Member

Clorith commented Jun 26, 2023

I do believe this falls a bit out of scope for the Health Check plugin (the intent of which is to be a troubleshooting tool, and not necessarily something that remains installed permanently, at least not now that the Site Health checks are part of WP core).

There are many security plugins that are aimed at this use case though, and I believe it is better to leave this behavior to them.

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

No branches or pull requests

3 participants