better cleanup of user files on user deletion#26792
Conversation
90951b5 to
a0359a1
Compare
blizzz
left a comment
There was a problem hiding this comment.
looks good – much better than before. did not test.
|
fyi @pierreozoux @acsfer |
Signed-off-by: Robin Appelman <robin@icewind.nl>
a0359a1 to
ed2d02d
Compare
|
The problem i was having was not on user deletion, but on files deletion. Does this PR helps in these cases? |
|
Should not this get backported, @icewind1991? |
|
It would be nice if it is backported :) But I also understand if it is not :) Thanks for fixing it, it is nice for the environment and user's privacy! |
|
/backport to stable21 |
|
My local storage is no longer getting cleaned up since this was merged ☹ Nextcloud 22.1.1 |
|
@metheos please open a new issue with the requested details (especially log from the delete action) |
Fix #9690
Handle cleanup of user data in the storage, not only the local fs.
Fixes user files not being deleted when using s3 primary storage.
to test:
On current master the files previously uploaded will be shown again, surviving the user delete.