Also delete objectstore data on user deletion#15147
Conversation
As there is no local data we have to explictly delete the data from the objectstore. Now this might time out if you do it via web. Maybe we should add a warning there if you do not use local storage? Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
|
|
||
| // Delete all files | ||
| if (!$userRoot->getStorage()->instanceOfStorage(Local::class)) { | ||
| $userRoot->delete(); |
There was a problem hiding this comment.
Should we just do that for all OS and then drop the lines 231 to 236?
There was a problem hiding this comment.
or should we do this in a background job?
There was a problem hiding this comment.
Yes that was also my consideration.
So then deleting the user is:
- Disabling the user
- Setting a special flag
- Scheduling backgroundjob
Because we need to fetch the actual user to do the deletion. Else we can't fetch the storage etc.
There was a problem hiding this comment.
But this is something for master. This can go in like that for the back ports and then we add the background job for master.
|
I'm fine with this for back porting and doing a background job and unified handling of this across all primary storages for master. |
|
CI fails on PHPUnit tests |
🏓 |
|
Actually the more I think about it the more it will most likely timeout. If you try to do this from the user management. Let me move this to 18 and see if we can soon come up with a proper backportable way that won't time out and do 💥 |
|
Is this already implemented in nc 17 and 18? As far as i can see, this bug still exists. Take a look at #9690 |
|
For me on NC 20 the issue is still present... |
IIPoliII
left a comment
IIPoliII
left a comment
There was a problem hiding this comment.
Works great but only using php-cli without timeout
6 participants
Fixes #9690
As there is no local data we have to explictly delete the data from the
objectstore.
Now this might time out if you do it via web. Maybe we should add a
warning there if you do not use local storage?
Signed-off-by: Roeland Jago Douma roeland@famdouma.nl