On Mon, 4 Feb 2002, Jost Krieger wrote:
> On Sun, Feb 03, 2002 at 01:29:31PM -0800, Craig Hughes wrote:
>> Yeah, I'd seen this claim of non-compliant headers in a few places
>> that seemed OK to me too -- The regex it's checking is pretty nasty
>> though. I'll see if I can figure out what jm was trying to do there
>> and fix it.
>
> Forgive me, I'm just a Newbie to spamassassin and have loosed it on 40000
> people on Friday.
> Run into the same problem, and it seems that
> a) RFC 822 mandated WS after the ":",
> b) RFC 2822 doesn't need the WS at all,
>
> so probably the line should be turned into
> } elsif (/^([^\x00-\x1f\x7f-\xff :]+):(.*)$/) {
Better that you do:
} elsif (/^([^\x00-\x1f\x7f-\xff :]+):\s*(.*)$/) {
or
} elsif (/^([^\x00-\x1f\x7f-\xff :]+):\s?(.*)$/) {
...depending on personal taste. That way the space is still removed if
it's present.
Daniel
--
Big whorls have little whorls that feed on their velocity
And little whorls have lesser whorls and so on to viscosity.
-- Lewis F. Richardson, quoted in Gleick's 'Chaos'
_______________________________________________
Spamassassin-talk mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/spamassassin-talk
