util_rb_3tld sites.uol.com.br - broken

[Mark Martinec <Ma...@ijs.si>]

> r1099780 | axb | 2011-05-05 15:24:23 +0200 (Thu, 05 May 2011) | 3 lines
> added: util_rb_3tld sites.uol.com.br

May  5 18:33:39.590 [54861] warn: config: SpamAssassin failed to parse line,
   "sites.uol.com.br" is not valid for "util_rb_3tld",
   skipping: util_rb_3tld sites.uol.com.br


The sites.uol.com.br has four fields,  util_rb_3tld allows three.

  Mark

Re: util_rb_3tld sites.uol.com.br - broken

[Mark Martinec <Ma...@ijs.si>]

> > The sites.uol.com.br has four fields,  util_rb_3tld allows three.

> thanks Mark
> removed.
> Guess I'll open a bug for this.

Thanks. While at it, it seems to me the regexp for util_rb_3tld is
also wrong for subsequent entries on the line: the first entry
is expected with three fields, the rest with two:

Mail/SpamAssassin/Conf.pm:

setting => 'util_rb_3tld',
[...]
unless ($value =~ /^[^\s.]+\.[^\s.]+\.[^\s.]+(?:\s+[^\s.]+\.[^\s.]+)*$/)

  Mark

Re: util_rb_3tld sites.uol.com.br - broken

[Yet Another Ninja <ax...@gmail.com>]

On 2011-05-05 18:44, Mark Martinec wrote:
>> r1099780 | axb | 2011-05-05 15:24:23 +0200 (Thu, 05 May 2011) | 3 lines
>> added: util_rb_3tld sites.uol.com.br
>
> May  5 18:33:39.590 [54861] warn: config: SpamAssassin failed to parse line,
>     "sites.uol.com.br" is not valid for "util_rb_3tld",
>     skipping: util_rb_3tld sites.uol.com.br
>
>
> The sites.uol.com.br has four fields,  util_rb_3tld allows three.
>
>    Mark

thanks Mark

removed.
Guess I'll open a bug for this.