1. Shouldn't we have Return-Path: higher up on the list of envelope
   sender headers that are checked in get_envelope_from() ?  I think the
   heuristic could probably use some work, perhaps look at the top
   Received: line to determine a priority order for headers.

2. ROUND_THE_WORLD - is this still a tflags net test?

3. NO_DNS_FOR_FROM is broken ... again.  I've known this has been
   misdesigned (using the foreground mx() function in Net::DNS) for a
   while, but the right solution eluded me until now.

   0.000   0.0000   0.0000    0.500   0.47    1.10  NO_DNS_FOR_FROM
   0.000   0.0000   0.0000    0.500   0.45    1.10  NO_DNS_FOR_FROM:bzoetekouw
   0.000   0.0000   0.0000    0.500   0.48    1.10  NO_DNS_FOR_FROM:jm
   0.000   0.0000   0.0000    0.500   0.47    1.10  NO_DNS_FOR_FROM:parkerm
   0.000   0.0000   0.0000    0.500   0.46    1.10  NO_DNS_FOR_FROM:quinlan
   0.000   0.0000   0.0000    0.500   0.49    1.10  NO_DNS_FOR_FROM:rODbegbie

  I have a patch to fix NO_DNS_FOR_FROM.  1000 ham and spam randomly
  sampled from my corpus after flushing my bind DNS cache:

   4.002   8.0000   0.0000    1.000   0.94    0.00 NO_DNS_FOR_FROM

  In addition to fixing the test and producing good results, the patch:

    - changes the MX test to use background sockets
    - skip_rbl_checks becomes skip_dns_checks, rbl_timeout becomes
      dns_timeout
    - check_mx_attempts and check_mx_delay are gone

Daniel

-- 
Daniel Quinlan
http://www.pathname.com/~quinlan/

Reply via email to