Skip to content

Conversation

Firefishy
Copy link
Member

Description

Add additional spam phrases to catch additional spam profiles.

How has this been tested?

TBC

@tomhughes
Copy link
Member

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.

@Firefishy Firefishy force-pushed the add-spam-keywords branch from f3acc9d to 71027c4 Compare May 23, 2025 14:37
@Firefishy Firefishy marked this pull request as ready for review May 28, 2025 15:22
@gravitystorm
Copy link
Collaborator

@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?

@tomhughes
Copy link
Member

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.

@pablobm
Copy link
Contributor

pablobm commented Oct 9, 2025

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?

@Firefishy
Copy link
Member Author

@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.

@tomhughes
Copy link
Member

It doesn't actually stop anything being created, they just get automatically suspended.

@tomhughes
Copy link
Member

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.

@pablobm
Copy link
Contributor

pablobm commented Oct 9, 2025

Keeping them in DB might be useful. For example if we want to use them to train a filter.

@Firefishy
Copy link
Member Author

It doesn't actually stop anything being created, they just get automatically suspended.

Sure. Any measure we take will not stop the spam, but this will make it a bit more difficult to successfully post spam.

@pablobm
Copy link
Contributor

pablobm commented Oct 9, 2025

we should just have a list that can be configured in settings

Agree. That's much better.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants