On Thu, 24 Jul 2003, Matt Kettler wrote:

> At 03:07 PM 7/24/2003 -0700, Abigail Marshall wrote:
> >I have one that I've been using for some time that activates
> >if the body contains a LINK to a .BIZ domain:
[snip..]
> I'd also suggest using \S* instead of .* after all, leading a regex off
> with .* really doesn't do anything. It requires there to be zero or more
> characters which aren't a newline. That rule would in fact match the string
> "if the body contains a LINK to a .BIZ domain" quite nicely.
>
> A further suggestion would be to require a dot, some non-whitespace
> characters, a dot, and biz.
>
> like this:
> uri BIZ_SITE    /\.\S*\.biz/i
> score BIZ_SITE 0.5

Minor nit, if you want to require "some non-whitespace characters"
then that should be \S+ not just \S*

However I would suggest omitting the requirment for two dots, as
I've seen plenty of: "http://best1netoffrs.biz/bno/offr...";
only one dot before the "biz"

I'd vote for:
     uri BIZ_SITE    /\.biz\//i

-- 
Dave Funk                                  University of Iowa
<dbfunk (at) engineering.uiowa.edu>        College of Engineering
319/335-5751   FAX: 319/384-0549           1256 Seamans Center
Sys_admin/Postmaster/cell_admin            Iowa City, IA 52242-1527
#include <std_disclaimer.h>
Better is not better, 'standard' is better. B{



-------------------------------------------------------
This SF.Net email sponsored by: Free pre-built ASP.NET sites including
Data Reports, E-commerce, Portals, and Forums are available now.
Download today and enter to win an XBOX or Visual Studio .NET.
http://aspnet.click-url.com/go/psa00100003ave/direct;at.aspnet_072303_01/01
_______________________________________________
Spamassassin-talk mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/spamassassin-talk

Reply via email to