Hi!
I'm not subscribed to the list, so I can't read replies without a cc: Also, please accept my apologies if this issue has already been discussed.
After checking out rc to a new site from svn some days ago (rev 631), I found an odd bug in the mail-header-list. Some of my contacts send mails containing an newline or carriage return in the From: header, thus breaking the _parse_address_list() in rcube_imap.inc. I'm not quite sure exactly which method is failing, but the result is that quotes are not removed, and the newline is converted to a html break inside the html anchor's href attribute, thus breaking the HTML validity.
I'm not sure if this newline is allowed in the From: header according to the RFCs, but it appeared.
Anyway: adding this single line patch to rcube_imap the problem disappeared from my installation.
Hopefully someone can use this knowledge to make RoundCube even better! Thanks!
.øs
Index: roundcubemail/program/include/rcube_imap.inc
--- roundcubemail/program/include/rcube_imap.inc (revision 361) +++ roundcubemail/program/include/rcube_imap.inc (working copy) @@ -2377,6 +2377,10 @@
function _parse_address_list($str) {
- // Remove any newlines and carriage returns from address list
- $str = preg_replace( "/[\r\n]/", " ", $str );
- $a = $this->_explode_quoted_string(',', $str); $result = array();