Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.4.7

Bug #591 Net_Sieve doesn't handle 'bye'
Submitted: 2004-01-16 10:00 UTC
From: Mark at 2pmtech dot co dot uk Assigned: damian
Status: Closed Package: Net_Sieve
PHP Version: Irrelevant OS: Irrelevant
Roadmaps: (Not assigned)    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 3 - 2 = ?

 
 [2004-01-16 10:00 UTC] Mark at 2pmtech dot co dot uk
Description: ------------ When Net_Sieve is communicating with a version of the Cyrus Timsieved that has the distributed 'murder' architecture enabled. If you connect to a timsieved for a user that the server does not host, the timsieved will responde with a 'bye' referral response that Net_Sieve does not see as finishing the conversation. The following patch fixes this problem --- Sieve.php-orig 2003-09-26 11:17:15.000000000 +0100 +++ Sieve.php 2003-09-26 11:17:30.000000000 +0100 @@ -526,10 +526,15 @@ $line .= str_replace("\r\n", ' ', $this->_sock->read($matches[1])); } return PEAR::raiseError(trim($response . substr($line, 2))); + } elseif ('bye' == strtolower(substr($line, 0, 3))) { + if (preg_match('/^bye \((referral) "([^"]+)/i', $line, $matches)) { + $line = $matches[1] . " " . $matches[2]; + } + return PEAR::raiseError(trim($response . $line)); } $response .= $line . "\r\n"; } } } ?>

Comments

 [2004-01-28 16:29 UTC] yunosh
A fixed version that also fixes bug #159 can be found at http://cvsweb.horde.org/cvs.php/framework/Net_Sieve/
 [2004-01-31 19:36 UTC] damian at php dot net
Thank you for your bug report. This issue has already been fixed in the latest released version of the package, which you can download at http://pear.php.net/packages.php Fixed in release 0.9