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

Bug #4794 drops protocol prefix, e.g. "ssl://" in referrals
Submitted: 2005-07-11 23:19 UTC
From: sky at columbia dot edu Assigned: amistry
Status: Closed Package: Net_Sieve
PHP Version: 4.3.11 OS: solaris
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 : 34 - 8 = ?

 
 [2005-07-11 23:19 UTC] sky at columbia dot edu
Description: ------------ While Net_Sieve doesn't support start TLS, we have hacked it to use SSL, by specifying the host as 'ssl://mailserver.tld' which PEAR does correctly. This fails when we use murder, since the referral drops the 'ssl://' and just directs to 'backend.tld' Even if it's a hack, it seems referrals should respect the original connection method. The following is a patch that fixes this for us: 948,949c948,951 < //$this->_data['host'] = $matches[1]; < $this->_data['host'] = $matches[2]; --- > > //replace the old host with the referral host preserving any protocol prefix > $this->_data['host'] = preg_replace('/\w+(?!(\w|\:\/\/)).*/',$matches[2], $this->_data['host']); >

Comments

 [2006-01-27 19:40 UTC] amistry at php dot net
Can you try the following patch to see if it fixes your problem? http://am-productions.biz/docs/Sieve.patch
 [2006-02-06 20:23 UTC] amistry at php dot net
I'm planning on updating the package sometime this week so if you're able to test the patch I can make sure your changes are integrated into the upcoming release.
 [2006-08-16 14:31 UTC] sky at columbia dot edu
yes, the patch fixes it.