← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands-website/anti_spam into lp:widelands-website

 

There is a akismet Pypi package, so we could easily add this to pip_requirements.txt.
 
> > 5) You are marking any post of a user with 'ji' in its name as SPAM. Is that
> > not a bit too broad?
> 
> Yes, maybe change it into  ' ji ' or ' ji'

Correcting: 'ji' is only used for topic names (subject). And i think it is better to have a difference between topic name and post text.

Looking at my gathered spam posts, we should also add ' molvi' to the keywords. Both in topic name and post text.

I will make the following changes tomorrow:
- Add ' molvi' to keywords
- Using of keyword lists in local_settings.py (instead of hardcoded in pybb/views.py)
- merge into trunk and deploy

For me this is quite an interesting problem and i would be interested to make an own 'anti_spam' app of it (at least because there is a lot to learn). But since akismet makes his/here job, for widelands it would be better to use this service, because it is maybe more effective in a shorter time. But if we use this we may need additional views and buttons to submit ham or spam to akismet.
-- 
https://code.launchpad.net/~widelands-dev/widelands-website/anti_spam/+merge/307869
Your team Widelands Developers is subscribed to branch lp:widelands-website.


References