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

Bug #4794 drops protocol prefix, e.g. "ssl://" in referrals
Submitted: 2005-07-11 18:19 UTC Modified: 2006-08-16 09:31 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  


 [2005-07-11 18: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 14: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 15: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 09:31 UTC] sky at columbia dot edu
yes, the patch fixes it.