Edgewall Software

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#12006 closed defect (worksforme)

TracError should not be raised due rejected submission (results in HTTPInternalErrors being logged at warn level) — at Version 5

Reported by: Ryan J Ollos Owned by: Dirk Stöcker
Priority: normal Milestone:
Component: plugin/spamfilter Version:
Severity: normal Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description (last modified by Ryan J Ollos)

In th:comment:1:ticket:11149, the following error was noted:

2013-06-06 09:45:38,814 Trac[main] WARNING: [218.6.15.69] HTTPInternalError: 500 Trac Error (<div class="message">Submission rejected as potential spam <ul><li>IP 218.6.15.69 blacklisted by list.blogspambl.com [2]</li></ul></div>)
2013-06-06 09:47:05,105 Trac[main] WARNING: [173.44.37.226] HTTPInternalError: 500 Trac Error (<div class="message">Submission rejected as potential spam <ul><li>Content contained these blacklisted patterns: '(?i)(?:diet|penis)[\W_]*(?:pills|enlargement)', '(?i)(?:[ck]asino|gambl(?:e|ing)|poker|(?:texas)?-?hold-?em|black-?jack|baccarr?at?|roulette|lott(?:o|ery)|jackpot|millionaires?|backgammon)', '(?i)&lt;a href=', '(?i)casino', '(?i)\[(?:url|link)=http://'</li><li>Maximum number of external links per post exceeded</li><li>SpamBayes determined spam probability of 88.56%</li></ul></div>)

I found more in the logs yesterday:

2015-03-26 07:27:13,838 Trac[api] WARNING: Del captcha_expected
2015-03-26 07:27:14,922 Trac[main] WARNING: [188.240.128.181] HTTPInternalError: 500 Trac Error (<div class="message">Submission rejected as potential spam <ul><li>Content contained these blacklisted patterns: '(?i)\.(?:blogspot|typepad|wordpress)\.com'</li><li>Register trap field starts with HTTP URL (https://flyfishingrodssite.wordpress.com/2015/ ...)</li><li>SpamBayes determined spam probability of 99.90%</li></ul></div>)

I'll investigate where we can modify the code to avoid raising a TracError.

Change History (5)

comment:1 by Dirk Stöcker, 9 years ago

Resolution: worksforme
Status: newclosed

That simply means that a submission was rejected finally. Normally a spammer get's captcha request and it dies there. But when he solves the captcha and rejection count is still high, than a 500 is issued by trac. Probably the default code for TracError I assume.

comment:2 by Ryan J Ollos, 9 years ago

I would like to consider changing the logging level so that the information is logged at debug level rather than warning.

Last edited 9 years ago by Ryan J Ollos (previous) (diff)

comment:3 by Dirk Stöcker, 9 years ago

This error is not logged by SpamFilter plugin.

comment:4 by Ryan J Ollos, 9 years ago

I have not looked yet. Why is SpamFilter plugin raising a TracError in this case? That is what may lead to some change needed for the plugin. Like I said in comment:description, I'll investigate, and retarget the ticket if needed.

Last edited 9 years ago by Ryan J Ollos (previous) (diff)

comment:5 by Ryan J Ollos, 9 years ago

Description: modified (diff)
Summary: HTTPInternalErrors in logTracError should not be raised due rejected submission (results in HTTPInternalErrors being logged at warn level)
Note: See TracTickets for help on using tickets.