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

Previews do not delete on file removal OR user deletion #17744

Closed
SimplyCorbett opened this issue Oct 29, 2019 · 1 comment
Closed

Previews do not delete on file removal OR user deletion #17744

SimplyCorbett opened this issue Oct 29, 2019 · 1 comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug

Comments

@SimplyCorbett
Copy link

SimplyCorbett commented Oct 29, 2019

Steps to reproduce

  1. Delete file from trash
  2. Run cron
  3. Check database
  4. Entry still exists
  5. Preview still exists
  6. Delete user.
  7. Run cron
  8. Entry still exists
  9. Preview still exists

Expected behaviour

Entry removed from database
Preview deleted

Actual behaviour

Entry not removed from database.
Preview is not deleted.

Expected longterm outcome

Users can rack up 100GB+ previews, sometimes in the TB if it's a very large account.
On a nextcloud install that has been running for a while this could completely fill your storage capacity. If this is not a S3 specific bug then your costs could skyrocket.

The database is no exception to this either as the size will only continue to grow.

Thoughts

I can't rule out this being due to S3 being the primary storage. But there are no errors to indicate this is a S3 issue.

Workarounds

There are no workarounds for this. S3 storage is not like regular storage where you can just delete the appdata folder.

This would require you to parse the database and create a script to delete the files manually and pray that it doesn't screw up.

I'm also not sure if the database being modified to remove the preview entries would have a negative outcome.

Server configuration

S3 primary
Operating system:
Debian 10
Web server:
Nginx
Database:
Mariadb
PHP version:
fpm7.3
Nextcloud version: (see Nextcloud admin page)
17 stable.
Updated from an older Nextcloud/ownCloud or fresh install:
Fresh install.

@SimplyCorbett
Copy link
Author

As of the latest beta all issues have been fixed on my end.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug
Projects
None yet
Development

No branches or pull requests

1 participant