Tickets :: Comment on [#12305] CalDAV user lookup bug

* Your Email Address
* Spam protection	Enter the letters below: .___..__.. , __ .__ \| \| \| \./ / `[__) \| \|__\ \| \__.\|
Comment	> 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
Attachment
Watch this ticket