http://bugzilla.spamassassin.org/show_bug.cgi?id=4176





------- Additional Comments From [EMAIL PROTECTED]  2005-05-09 13:56 -------
Subject: Re:  [review] RFE: please add pattern for nate.com redirector

> (btw -- not as any kind of block here, but can anyone explain something to me 
> --
> about the redirector_pattern lines:
> 
> redirector_pattern     /^(?:http://)?chkpt\.zdnet\.com/chkpt/\w+/(.*)$/
> 
> why is '(?:http://)?' conditional?  should we not just ensure that any input 
> to
> that match will always be a full URL with protocol, and leave off the
> conditionality?

I wrote the regexp before I wrote the code to do the matching and found 
out that the protocol would already be added if missing.  Will fix in a 
second.





------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
You are on the CC list for the bug, or are watching someone who is.

Reply via email to