https://issues.apache.org/SpamAssassin/show_bug.cgi?id=7070

Quanah Gibson-Mount <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|INVALID                     |---

--- Comment #4 from Quanah Gibson-Mount <[email protected]> ---
The documentation notes that the default is:

.2 * max.

We can see that 45 * .2 = 9

So that's where it is getting the 9 second minimum from.  I.e., the code and
the documentation do not match.

You can find this in Conf.pm:

3328     default => 15,

We can see rbl_timeout_min gets set here:

3347       if (!defined $zone) {  # a global setting
3348         $self->{rbl_timeout}     = 0 + $timeout;
3349         $self->{rbl_timeout_min} = 0 + $timeout_min  if defined
$timeout_min;
3350       }

Now, if we look in AsyncLoop.pm, we see:

213   my $t_end = $ent->{timeout_min};       # application-specified has
precedence
214   $t_end = $settings->{rbl_timeout_min}  if $settings && !defined $t_end;
215   $t_end = 0.2 * $t_init  if !defined $t_end;
216   $t_end = 0  if $t_end < 0;  # just in case
217   $t_init = $t_end  if $t_init < $t_end;


Apparently, $t_end is not correctly getting set when rbl_timeout_min is
specified.

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to