Description: ------------ One Roundcube user has a problem using Net_Sieve with DBMail-timesieved. Please, tak a look at http://www.roundcubeforum.net/10-miscellaneous/33-roundcube-discussion/9168-managesieve-plugin-fail-dbmail-timsieved.html (proposed solution in first post).

Has the log been posted anywhere yet?

No. The check ($lastline === '') doesn't looks good to me (especially when you have rtrim($lastline) before). I have removed it and for now it's working with dovecot.

On error Net_Socket's fgets() method returns pear error object or FALSE. So, I propose to move the check before rtrim() and check for false not empty string.

Also work with Cyrus. I went thru commit log and found that the lines were added (uncommented) to fix http://pear.php.net/bugs/bug.php?id=8452. So, maybe checking for FALSE would be a good solution for both. The issue was requested again here http://trac.roundcube.net/ticket/1488594. I'll provide debug log if I get any.

It looks we've found a solution/workaround. The reason of the issue is a bug in DBMail http://dbmail.org/mantis/view.php?id=963. It's already fixed but not released. I've got a simple patch which workarounds this issue: --- a/plugins/managesieve/lib/Net/Sieve.php +++ b/plugins/managesieve/lib/Net/Sieve.php @@ -1098,7 +1098,7 @@ class Net_Sieve return PEAR::raiseError(trim($response . $line), 6); } - if (preg_match('/^{([0-9]+)}/i', $line, $matches)) { + if (preg_match('/^{([0-9]+)\+?}/i', $line, $matches)) { // Matches literal string responses. $line = $this->_recvBytes($matches[1] + 2);

This had been fixed almost two years ago and not been released since then?? I consider DBMail dead.