|
From: SourceForge.net <no...@so...> - 2011-10-07 22:59:05
|
Bugs item #3420440, was opened at 2011-10-07 23:06 Message generated for change (Comment added) made by christian_boltz You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3420440&group_id=191583 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: SVN (please specify revision!) >Status: Closed >Resolution: Fixed Priority: 5 Private: No Submitted By: Christian Boltz (christian_boltz) Assigned to: Nobody/Anonymous (nobody) Summary: gen_show_status POP/IMAP check broken - recipient_delimiter Initial Comment: from #postfixadmin [22:14] <stderr1> cboltz: I think I found a bug in functions.php:gen_show_status() [22:15] <cboltz> details please ;-) [22:15] <stderr1> cboltz: haven't traced it but from reading the code I'm quite sure the POP/IMAP check incorrectly cuts the part after the recipient delimiter [22:15] <cboltz> 2.3.x or trunk? [22:16] <stderr1> In the deliverable check it's correct because it works on one address at a time, but in the pop/imap one the preg_replace works on the whole comma separated list [22:16] <stderr1> 2.3.5 if I'm not mistaken, wait... [22:16] <stderr1> no, 2.3.3 [22:17] <stderr1> so fo...@ba...,bl...@so... would be cut to fo...@so... [22:19] <cboltz> indeed, this gives me "forward only" [22:19] <cboltz> same in trunk ---------------------------------------------------------------------- >Comment By: Christian Boltz (christian_boltz) Date: 2011-10-08 00:59 Message: It turned out that it's not a bug in gen_show_status(), but in list-virtual. list-virtual has its own check to test if the target is the mailbox, and this check did not honor the recipient delimiter. Fixed in SVN r1198 (trunk) and 1199 (2.3 branch). ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3420440&group_id=191583 |
