-
Notifications
You must be signed in to change notification settings - Fork 1k
Add additional common SPAMMY_PHRASES #6044
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
base: master
Are you sure you want to change the base?
Conversation
I've no idea what I was thinking when I added that blacklist in 12d3f94 but I really don't want to be going down the route of playing whackamole in this repository. |
f3acc9d
to
71027c4
Compare
@tomhughes I agree that we don't want to go down this route, so I'm minded to decline this PR. Do we also want to remove the existing list of phrases, to simplify the code? |
Apparently I added that in 12d3f94 but I have no memory of it... No doubt it was targeting some specific SEO operation at that time so is likely not doing much now. |
While I agree that this route is not desirable in the long term, I have to ask: do we have a better solution at the moment? @Firefishy: would this patch at least alleviate the current workload from spam moderation? Are these phrases still reflective of the latest waves of spam? |
Yes, this will help to reduce the 100s of spam accounts that are created each day. This isn't the ultimate solution, but an intermediate measure. It should catch around 15% of the spam accounts. |
It doesn't actually stop anything being created, they just get automatically suspended. |
At least very least I don't think the list of phrases should be baked into the source, we should just have a list that can be configured in settings which can be updated by chef in the production deployment. |
Keeping them in DB might be useful. For example if we want to use them to train a filter. |
Sure. Any measure we take will not stop the spam, but this will make it a bit more difficult to successfully post spam. |
Agree. That's much better. |
Description
Add additional spam phrases to catch additional spam profiles.
How has this been tested?
TBC