Tickets :: Comment on [#7881] Driver.php,v 1.46

* Your Email Address
* Spam protection	Enter the letters below: .__.. . __.. . . \| \|\|__\|(__ \| \| \| \|__\|\| \|.__)\|/\\|\__\|
Comment	> There si a little bug when you do not set up any realm and use full > user names like: > > info@example.com > > > > you must set up the $params correctly before you check the hordeauth. > > here is a patch, just need change the lines: > > > > The original one: > > function Vacation_Driver($user, $realm, $params = array()) > > { > > $this->_params = $params; > > > > // Check if hordeauth is set to 'full' > > $hordeauth = $this->getParam('hordeauth'); > > if ($hordeauth !== 'full') { > > @list($user,) = explode('@', $user, 2); > > } > > $this->_user = $user; > > > > if (!isset($params[$realm])) { > > $realm = 'default'; > > } > > $this->_realm = $realm; > > } > > > > The fixed one: > > function Vacation_Driver($user, $realm, $params = array()) > > { > > $this->_params = $params; > > > > if (!isset($params[$realm])) { > > $realm = 'default'; > > } > > $this->_realm = $realm; > > > > // Check if hordeauth is set to 'full' > > $hordeauth = $this->getParam('hordeauth'); > > if ($hordeauth !== 'full') { > > @list($user,) = explode('@', $user, 2); > > } > > $this->_user = $user; > > }
Attachment
Watch this ticket