Tickets :: [#9736] Missing db parameter in CyrSQL driver

6.0.0-beta6

4/8/26

Summary	Missing db parameter in CyrSQL driver
Queue	Horde Framework Packages
Queue Version	Git master
Type	Bug
State	Resolved
Priority	2. Medium
Owners	slusarz (at) horde (dot) org
Requester	vilius (at) lnk (dot) lt
Created	3/28/11 (5490 days ago)
Due
Updated	3/29/11 (5489 days ago)
Assigned	3/28/11 (5490 days ago)
Resolved	3/29/11 (5489 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

03/29/2011 07:27:22 PM	vilius (at) lnk (dot) lt	Comment #7	Reply to this comment
Sorry wrong file version. Now I get another bug http://bugs.horde.org/ticket/9741

03/29/2011 07:07:18 PM	Michael Slusarz	Comment #6	Reply to this comment
2011-03-29T21:53:53+03:00 NOTICE: HORDE PHP ERROR: Undefined variable: orig_params [pid 14582 on line 119 of "/usr/share/pear/Horde/Core/Factory/Auth.php"] Line 119 of Factory/Auth.php is: 'secure' => ($params['secure'] == 'none') ? null : $params['secure'], There is no '$orig_params' on that line (or anywhere near that line for that matter).

03/29/2011 06:56:38 PM	vilius (at) lnk (dot) lt	Comment #5	Reply to this comment
I'm not sure, but the fix breaks IMP login in traditional mode? Dynamic mode works fine. I get: 2011-03-29T21:53:53+03:00 NOTICE: HORDE PHP ERROR: Undefined variable: orig_params [pid 14582 on line 119 of "/usr/share/pear/Horde/Core/Factory/Auth.php"] 2011-03-29T21:53:53+03:00 DE~~BUG: 1~~. Horde_Registry::appInit() /var/www/html/horde4/imp/index.php:16 2. Horde_Registry->pushApp() /usr/share/pear/Horde/Registry.php:238 3. Horde_Registry->checkExistingAuth() /usr/share/pear/Horde/Registry.php:1259 4. Horde_Core_Factory_Auth->create() /usr/share/pear/Horde/Registry.php:2169 5. Horde::errorHandler() /usr/share/pear/Horde/Core/Factory/Auth.php:119

03/29/2011 05:28:30 PM	Michael Slusarz	State ⇒ Resolved

03/29/2011 05:24:23 PM	vilius (at) lnk (dot) lt	Comment #4	Reply to this comment
Yes, thank you.

03/28/2011 10:08:45 PM	Michael Slusarz	Comment #3 State ⇒ Feedback Assigned to Michael Slusarz	Reply to this comment
Does this fix things?

03/28/2011 10:08:28 PM	Git Commit	Comment #2	Reply to this comment
Changes have been made in Git for this ticket: ~~Bug #9736~~: Pass DB parameter to Cyrsql driver 1 files changed, 10 insertions(+), 0 deletions(-) http://git.horde.org/horde-git/-/commit/aeed8257c5fd04470ecbfd75618e8df331880e9f

03/28/2011 09:02:16 PM	vilius (at) lnk (dot) lt	Comment #1 Priority ⇒ 2. Medium Type ⇒ Bug Summary ⇒ Missing db parameter in CyrSQL driver Queue ⇒ Horde Framework Packages Milestone ⇒ Patch ⇒ No State ⇒ Unconfirmed	Reply to this comment
Cyrus SQL driver doesn't work and throws exception below. Ordinary Sql.php driver with the same configuration parameters works just fine. EMERG: HORDE [horde] Missing db parameter. [pid 13742 on line 80 of "/usr/share/pear/Horde/Auth/Sql.php"] DE~~BUG: 1~~. Horde_Registry->isAuthenticated() /var/www/html/horde4/login.php:60 2. Horde_Core_Factory_Auth->create() /usr/share/pear/Horde/Registry.php:1807 3. Horde_Auth::factory() /usr/share/pear/Horde/Core/Factory/Auth.php:156 4. Horde_Auth_Cyrsql->__construct() /usr/share/pear/Horde/Auth.php:68 5. Horde_Auth_Sql->__construct() /usr/share/pear/Horde/Auth/Cyrsql.php:150 Using Horde 4 RC1.