-
Notifications
You must be signed in to change notification settings - Fork 281
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
Delete boards that users own once they are deleted #1976
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1976 +/- ##
==========================================
- Coverage 56.92% 56.81% -0.11%
==========================================
Files 77 77
Lines 3842 3849 +7
==========================================
Hits 2187 2187
- Misses 1655 1662 +7 |
Will this solution cover also boards that has been shared with the "deleted" user? |
Yes, user acl entries already get removes some lines above https://github.com/nextcloud/deck/blob/master/lib/AppInfo/Application.php#L120 |
Signed-off-by: Julius Härtl <jus@bitgrid.net>
000607d
to
1f4ab91
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
Sorry for nut picking... but maybe you should also add a clean step on app upgrade which check for cases where boards are present but user has been deleted, just to make sure? It's still potential leaked data... |
Make sure that users boards are properly removed on user delete