Skip to content
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

Reports spam if only ONE field matches with stopforumspam #1

Open
phred opened this issue May 20, 2013 · 1 comment
Open

Reports spam if only ONE field matches with stopforumspam #1

phred opened this issue May 20, 2013 · 1 comment

Comments

@phred
Copy link

phred commented May 20, 2013

Thanks for this plugin, it's already cut down on our wiki spam!

This was dumping one me to the spam error page during login. The plugin requests this from StopForumSpam:
http://www.stopforumspam.com/api?username[]=Fred&email[]=fred.alger%40foxycart.com&ip[]=69.1.26.101&&unix

Which gives this response:

     <root>
          <success>1</success>
          <username>
               <value>Fred</value>
               <lastseen>1368463209</lastseen>
               <frequency>95</frequency>
               <appears>1</appears>
          </username>
          <email>
               <value>[email protected]</value>
               <frequency>0</frequency>
               <appears>0</appears>
          </email>
          <ip>
               <value>69.1.26.101</value>
               <frequency>0</frequency>
               <appears>0</appears>
          </ip>
     </root>

The issue is that "Fred" is a common spam account name. However, it's also a common firstname, my first name in fact 😄. My IP and email aren't used for spam, and I shouldn't be blocked from logging in (especially as site admin ❗)

I manually hacked my plugin to check if $isSfsSpam > 1 so that we can get back to work. Can do a pull request if you'd like.

@michield
Copy link
Owner

You can use the whitelist setting to avoid being locked out. But I see your point....

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

No branches or pull requests

2 participants