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...,blurb@some.where would be cut to foo@some.where
[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
|