ProjectLighthouse icon indicating copy to clipboard operation
ProjectLighthouse copied to clipboard

Purge Comments Not Working Perfectly

Open m88youngling opened this issue 1 year ago • 6 comments

Describe the bug Using the recent purge comments feature from be11e138f032ec306f9c732b4a3a3cee47740edc fails to delete all comments from a user. Most comments will be deleted but some will remain.

To Reproduce Steps to reproduce the behavior:

  1. Find a user spamming your server
  2. Click the button to delete all their comments
  3. (Optional) the user is banned when I observe the behavior.

Expected behavior All comments belonging to the user should be removed, not most of them.

Screenshots The only examples I have are not appropriate for this issue.

m88youngling avatar Jun 21 '24 17:06 m88youngling

i'd suggest just to remove all comments made from the same IP

skittlez1250 avatar Jun 21 '24 23:06 skittlez1250

That would require storing IP addresses in the database again, which we haven't decided is a good idea or not.

m88youngling avatar Jun 22 '24 03:06 m88youngling

that's kinda a weird thought, but what i think is happening here is that it actually deletes all the comments that existed at the moment the action was called and since the user is constantly spamming new comments those new ones are not getting deleted, so technically it works as intended it's just the way it works is not effective against spamming

jackcaver avatar Jun 25 '24 06:06 jackcaver

i'd suggest just to remove all comments made from the same IP

this is also quite useless i think, since the spammers could have a lot of different IPs and multiple good people could have the same IP as a bad guy, so this could delete all of the comments of innocent people

jackcaver avatar Jun 25 '24 06:06 jackcaver

that's kinda a weird thought, but what i think is happening here is that it actually deletes all the comments that existed at the moment the action was called and since the user is constantly spamming new comments those new ones are not getting deleted, so technically it works as intended it's just the way it works is not effective against spamming

I think you are probably right. In that case, best to just click the button multiple times. It might be best to close this issue if that's the case

m88youngling avatar Jun 25 '24 10:06 m88youngling

that's kinda a weird thought, but what i think is happening here is that it actually deletes all the comments that existed at the moment the action was called and since the user is constantly spamming new comments those new ones are not getting deleted, so technically it works as intended it's just the way it works is not effective against spamming

I think you are probably right. In that case, best to just click the button multiple times. It might be best to close this issue if that's the case

I think that a better solution to that is disabling user's ability to add comments and then remove all

jackcaver avatar Jun 25 '24 10:06 jackcaver