Ping?

Shouldn't this rather be done through addcslashes() like anywhere else 
in the maildrop driver?
I consider this as a "yes."

The extra trims in the foreach loop are from paranoia about too many 
exploits using embedded control characters in mail headers.
I redid the ignorelist regex to simplify it for debugging my code.  It 
can certainly be bulked-up, but I think the original ignorelist is not 
workable in its current form.  I am using it now as it sits.

What does the extra trimming of the email addresses do and why?

What's about the currently outcommented ignorelist stuff a few lines 
above? Can this go now?

Here is the latest diff, including start and end times, 
list/bulk/junk/admin mail ignoring, and excluded addresses.  I 
simplified the admin mail regexp for debugging purposes.

Please provide an updated patch.

After you have rendered judgement on these, I have suggestions for 
handling exclusions as well

please move the "current_time=time" line before the vacation start/end 
time checks

Here is the latest diff against CVS HEAD, including 'ignorelist' checking.



In the future, I will do more testing before posting.

Heavier testing led to the addition to an else clause at the end, so 
the generated reply is not delivered to the local mailbox



=== add after line 518 ===

             $this->_action[] = '  else';

             $this->_action[] = '  {';

             $this->_action[] = '    to "/dev/null";';

             $this->_action[] = '  }';

This has been lightly tested.