[Tickets #5816] Re: Added more tests to maildrop Script and fixed REJECT action

[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=5816
-----------------------------------------------------------------------
 Ticket             | 5816
 Updated By         | Chuck Hagenbuch <chuck at horde.org>
 Summary            | Added more tests to maildrop Script and fixed REJECT action
 Queue              | Ingo
 Version            | HEAD
 Type               | Enhancement
-State              | Resolved
+State              | Feedback
-Priority           | 1. Low
+Priority           | 2. Medium
 Owners             | Chuck Hagenbuch
-----------------------------------------------------------------------


Chuck Hagenbuch <chuck at horde.org> (2007-11-20 12:47) wrote:

> I recently noticed why 'contains' was used so far... because the 
> From: header mostly consists of more than just the sender address. 
> Using 'is' surely isn't the right solution, the proper way to do this 
> is using maildrops getaddr() on the From header and checking each 
> listed email address...
> I'll take a shot at this when dealing with the changes I mentioned in 
> bug 5843.
>
> If a release is due, maybe you should switch back to 'contains' for 
> time being. Otherwise just wait for my patch ;)

We're near an RC, but not a final release. There should really not be
multiple addresses in From:, but I see the point about getaddr. I'll wait
for (or look into myself, though that might be wishful) a patch for now.

Thanks for all the help on maildrop support!