Sat, 05 Jul 2008 19:42:33 -0400 http://bugs.horde.org/ticket/5912 set start and stop times for maildrop vacation recipes This has been lightly tested. Wed, 21 Nov 2007 11:41:27 -0500 http://bugs.horde.org/ticket/5912#t38874 > This has been lightly tested. 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[] = ' }'; Wed, 21 Nov 2007 12:40:41 -0500 http://bugs.horde.org/ticket/5912#t38876 Here is the latest diff against CVS HEAD, including 'ignorelist' checking. In the future, I will do more testing before posting. Wed, 21 Nov 2007 13:36:06 -0500 http://bugs.horde.org/ticket/5912#t38886 please move the "current_time=time" line before the vacation start/end time checks Wed, 21 Nov 2007 14:44:25 -0500 http://bugs.horde.org/ticket/5912#t38888 After you have rendered judgement on these, I have suggestions for handling exclusions as well Wed, 21 Nov 2007 15:02:19 -0500 http://bugs.horde.org/ticket/5912#t38895 > please move the "current_time=time" line before the vacation > start/end time checks Please provide an updated patch. Wed, 21 Nov 2007 18:39:21 -0500 http://bugs.horde.org/ticket/5912#t38902 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. Mon, 26 Nov 2007 10:36:23 -0500 http://bugs.horde.org/ticket/5912#t39032 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? Mon, 03 Dec 2007 12:52:22 -0500 http://bugs.horde.org/ticket/5912#t39302 > 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. > 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. Mon, 03 Dec 2007 13:38:52 -0500 http://bugs.horde.org/ticket/5912#t39304 >> 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." Fri, 07 Dec 2007 09:52:30 -0500 http://bugs.horde.org/ticket/5912#t39584 Ping? Sun, 24 Feb 2008 07:34:57 -0500 http://bugs.horde.org/ticket/5912#t42834