-
Notifications
You must be signed in to change notification settings - Fork 262
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
Some files have not passed the integrity check. #9731
Comments
Do you happen to have cms_pico installed? It's also possible it's another app that uses the same dependency. Just dealt with this on the forum: The |
I do have pico_cms enabled, yes. |
Then this can only be fixed by cms_pico. They're using the cache in HTMLPurifier without setting the path for it so it's defaulting to "wherever the library is installed": http://htmlpurifier.org/live/configdoc/plain.html
|
HTMLSerializer uses the path the library is installed it as the cache by default. Since other Nc apps use HTMLSerialize, the app who's copy of HTMLSerialize happens to get used first, ends up with a bunch of stray cache files. Besides being messy, this brings things like integrity checks. See nextcloud/mail#9731 There are three choices here: - disable the cache (this what the Nc `mail` app does) - leave it enabled by specifying a path - accept it as is Performance impact of disabling is unknown. If we eventually want this enabled still in Pico, we can set `Cache.SerializePath` to somewhere we're comfortable saving data (locally on the instance...). Ref: http://htmlpurifier.org/live/configdoc/plain.html#Cache.DefinitionImpl Signed-off-by: Josh <josh.t.richards@gmail.com>
nextcloud/cms_pico#248 should address this. You'll also need to delete the extra files shown in a recent integrity report. |
Steps to reproduce
Expected behavior
Integrity check should pass clean
Actual behavior
Integrity check reports extra files. Manually deleting helps but they are back at least after an update of Mail,
Mail app version
3.7.1
Mailserver or service
postfix/cyrus
Operating system
Linux/docker
PHP engine version
PHP 8.2
Web server
Nginx
Database
MySQL
Additional info
The text was updated successfully, but these errors were encountered: