CalDAV user lookup bug

I have triggered an error when using the new CalDAV function

Wed, 05 Jun 2013 14:58:08 +0000

I have triggered an error when using the new CalDAV functionality of kronolith. If using CalDAV and authentication via IMP, imp's _init() does not get called in the function mentioned later. This leaves the values 'add', 'remove', and 'list' in the capability list even if not configured in the driver. framework/Dav/lib/Horde/Dav/Principals.php, function getPrincipalByPath() erroneously assumes that $this->_auth->hasCapability('list') is true and exits with an user does not exist exception. This way I am unable to use CalDAV. My suggestions to fix this: Option 1 (preferred, patch attached): Instead of removing the "add", "list" and "remove" capabilities in imp/lib/Application.php when they are not supported, merge them in _init() if they are supported Option 2: call imp's _init() before checking the problem in Dav/Principals.php Option 3: modify the check in Dav/Principals.php to accept such a situation

> Option 2: > call imp's _init() before checking the proble

Thu, 13 Jun 2013 12:15:01 +0000

> Option 2: > call imp's _init() before checking the problem in Dav/Principals.php I have no idea what is going on in this Dav code, but anything that is trying to access an application's Application class without initializing it is very broken.

It's not calling the Application API at all, IMP is only inv

Thu, 13 Jun 2013 13:45:18 +0000

It's not calling the Application API at all, IMP is only involved via authentication: http://git.horde.org/co.php/framework/Dav/lib/Horde/Dav/Auth.php?rt=horde-git&r=fd567b12209dfde32d049dbb467578709dbb9fbf with $_auth being $injector->getInstance('Horde_Core_Factory_Auth')->create()

I can reproduce this now. In Horde_Core_Auth_Application:

Thu, 13 Jun 2013 14:02:19 +0000

I can reproduce this now. In Horde_Core_Auth_Application::listUsers(), the first call to hasCapability('list') returns true, so $registry->callAppMethod($this->_app, 'authUserList') is called. This goes down to Horde_Auth_Imap::listUsers() where hasCapability('list') now returns false.

Changes have been made in Git (master): commit 808dbc224ab6

Thu, 13 Jun 2013 14:16:44 +0000

Changes have been made in Git (master): commit 808dbc224ab677a614884d5464d75d440dbc35b3 Author: Jan Schneider Date: Thu Jun 13 16:16:16 2013 +0200 Detect admin capabilities in _bootstrap() instead of _init() (Bug #12305). imp/docs/CHANGES | 1 + imp/lib/Application.php | 18 ++++++++++-------- imp/package.xml | 2 ++ 3 files changed, 13 insertions(+), 8 deletions(-) http://git.horde.org/horde-git/-/commit/808dbc224ab677a614884d5464d75d440dbc35b3

I discover some problem after employing hook (authusername)

Tue, 09 Jul 2013 17:08:31 +0000

I discover some problem after employing hook (authusername) to strip off domain name when loggin horde and kronolith (4.1.1). When I try using kronolith and obtain the subscription URL of CalDAV, it shows: https://mail.domain.com/rpc/principals/foo@domain.com/ Then, I try adding the account through iCal (OSX10.8.4) with using the URL and username as 'foo@domain.com' and it shows 'Authentication failed. Your username and password were rejected by the server'. The horde logging shows nothing about it. The weird point lies in when I using the following URL to add account: https://mail.domain.com/rpc/principals/foo/ and using 'foo' as username, iCal can add the account successfully. However, there is simply no calendar shown. On the contrary, if I disable the hook.php and caldav in horde work flawlessly with horde. I have two calendars in horde site and they are shown in iCal. When I enable the hook.php again and re-add the account again in ical no calendar can be shown. Therefore, I believe there are some problems in using the hook and CalDAV user name. Kinglok, Fong

Extending Jan's change to kronolith/lib/Form/EditCalendar.ph

Wed, 06 Aug 2014 07:45:39 +0000

Extending Jan's change to kronolith/lib/Form/EditCalendar.php seems to fix this, see attached patchfile. Many thanks to Jan for fixing this issue, it was a major request from our users. Regards, Andreas

Sorry, wrong ticket, was meant for #12380 > Extending Jan

Wed, 06 Aug 2014 07:47:47 +0000

Sorry, wrong ticket, was meant for #12380 > Extending Jan's change to kronolith/lib/Form/EditCalendar.php seems > to fix this, see attached patchfile. > > Many thanks to Jan for fixing this issue, it was a major request from > our users. > > Regards, > Andreas