Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.4.8
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 : 8 + 20 = ?

 
 [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] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send
MANUAL CONFIRMATION IS NOT POSSIBLE.  Write a message to pear-dev@lists.php.net
to request the confirmation link.  All bugs/comments/patches associated with this

email address will be deleted within 48 hours if the account request is not confirmed!
 [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