Remove unused avatars

[BaxAndrei]

Abbility to remove unused avatars.

[bagasme]

@BaxAndrei any details?

[BaxAndrei]

I noticed that when a user changes their avatar, a new one will be uploaded to the server, and the old one will still be there. I also know that the name of the avatar is a hash of it. My proposal is that, if a user changes his avatar, the system will check in the database if there is any user using the old avatar, and if it is no longer used by anyone, it will be automatically deleted.

[guillep2k]

There's already a PR working on that by adding the user's ID as input for the hash, so there will never be a collision.

[guillep2k]

@lunny Maybe a check for unused resources can be added to the "doctor" (avatars, attachments, LFS objects)?

[BaxAndrei]

@lunny Maybe a check for unused resources can be added to the "doctor" (avatars, attachments, LFS objects)?

Yes, will help.

[lunny]

@guillep2k good idea.