https://issues.apache.org/SpamAssassin/show_bug.cgi?id=7070
Kevin A. McGrail <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|REOPENED |RESOLVED Resolution|--- |FIXED --- Comment #14 from Kevin A. McGrail <[email protected]> --- (In reply to Quanah Gibson-Mount from comment #13) > Ah, I found the bug. For rbl_timeout, we correctly fall back to the config > setting, if it isn't set with $ent or $settings: > > my $t_init = $ent->{timeout_initial}; # application-specified has > precedence > $t_init = $settings->{rbl_timeout} if $settings && !defined $t_init; > $t_init = $self->{main}->{conf}->{rbl_timeout} if !defined $t_init; > $t_init = 0 if !defined $t_init; # last-resort default, just in case > > This is missing for rbl_timeout_min. I already pointed out the exact logic difference and why it wasn't used. The point is I don't know if it is a bug or intended functionality. Should the rbl_timeout_min only apply in a zone? It was added in 2007 and http://www.gossamer-threads.com/lists/spamassassin/commits/110230. But at this point you've beaten me down and I'll add it to trunk barring someone's objection that it will break something intended otherwise. And next time, please discuss this on dev list not bugzilla. -- You are receiving this mail because: You are the assignee for the bug.
