Bug #4794

drops protocol prefix, e.g. "ssl://" in referrals

Details

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)

Comments

[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']);
>

[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.