#8121 closed enhancement (fixed)
[PATCH] Show matching regular expressions
Reported by: | Owned by: | ||
---|---|---|---|
Priority: | normal | Milestone: | not applicable |
Component: | plugin/spamfilter | Version: | none |
Severity: | minor | Keywords: | |
Cc: | Branch: | ||
Release Notes: | |||
API Changes: | |||
Internal Changes: |
Description
The attached patch updates the regex filter such that if submitted content gets blocked because of matching patterns it will list those patterns.
I wrote it while investigating reports from users of trac-hacks.org saying that (harmless / non-spamy) submissions were blocked because of some blacklisted pattern. Since our BadContent page contains ~1800 patterns, parsing them mentally to find the culprit was actually no option.
The patch probably requires some more love before it can be committed upstream, but it may at least serve as a prototype.
Attachments (1)
Change History (6)
by , 16 years ago
Attachment: | show-matching-patterns.diff added |
---|
follow-up: 2 comment:1 by , 16 years ago
Milestone: | → not applicable |
---|
comment:2 by , 16 years ago
Replying to cboos:
Michael, as you're an expert "Spam fighter" if I recall correctly,
I'd not call me an expert in that regard, but due to authoring ScallyWhack I have at least some experience with fighting spam that hits Trac instances. :)
would you like to participate to the maintenance of the SpamFilter?
Yes, that would be great.
comment:3 by , 15 years ago
Summary: | Show matching regular expressions → [PATCH] Show matching regular expressions |
---|
Tested this for some month now and found it very useful!
P.S. I'm the same anon as in #8257.
comment:4 by , 15 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Patch applied as is in r8328, as others found it useful.
Michael, if you'd like to refine this further, please do so!
comment:5 by , 15 years ago
Owner: | set to |
---|
Sounds useful.
Michael, as you're an expert "Spam fighter" if I recall correctly, would you like to participate to the maintenance of the SpamFilter? (otherwise that patch might never get applied…)