[Tickets #5912] Re: set start and stop times for maildrop vacation recipes

[bugs at bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/?id=5912
-----------------------------------------------------------------------
 Ticket             | 5912
 Updated By         | Jan Schneider <jan at horde.org>
 Summary            | set start and stop times for maildrop vacation recipes
 Queue              | Ingo
 Version            | HEAD
 Type               | Enhancement
 State              | Feedback
 Priority           | 1. Low
 Owners             | 
-----------------------------------------------------------------------


Jan Schneider <jan at horde.org> (2007-12-07 06:52) wrote:

>> What does the extra trimming of the email addresses do and why?
>
> The extra trims in the foreach loop are from paranoia about too many 
> exploits using embedded control characters in mail headers.

Shouldn't this rather be done through addcslashes() like anywhere else in
the maildrop driver?

>> What's about the currently outcommented ignorelist stuff a few lines
>> above? Can this go now?
>
> 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.

I consider this as a "yes."