#263 gen_show_status POP/IMAP check broken - recipient_delimiter

closed-fixed
nobody
None
5
2013-12-01
2011-10-07
No

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 foo+bar@baz.com,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

Discussion

  • Christian Boltz

    Christian Boltz - 2011-10-07

    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).

     
  • Christian Boltz

    Christian Boltz - 2011-10-07
    • status: open --> closed-fixed
     

Log in to post a comment.