View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0002547 | SOGo Connector | with external server | public | 2013-12-13 08:55 | 2016-07-21 20:11 |
Reporter | Sven Giermann | Assigned To | ludovic | ||
Priority | normal | Severity | feature | Reproducibility | always |
Status | closed | Resolution | suspended | ||
Product Version | Monotone / nightly | ||||
Summary | 0002547: SOGo Connector should also handle CardDAV responses with additional header lines before "BEGIN:VCARD" | ||||
Description | After applying the patch so solve the problems reported in Bug 1411, I finally came back to productive use of SOGo Connector CardDAV address book against my Kerio Connect server... There's only one change still needed to get it work with Kerio Connect: The attached patch will simply change this logic to look for the first occurence of "begin:vcard" in the response and to use this to import the VCARD or VLIST. It hopefully does not interfere with any product other than Kerio Connect ;) | ||||
Tags | No tags attached. | ||||
skip_headers.diff (1,349 bytes)
--- chrome/content/sogo-connector/general/sync.addressbook.groupdav.js Fr 22. Nov 21:31:04 2013 +++ chrome/content/sogo-connector/general/sync.addressbook.groupdav.js Mo 9. Dez 16:32:08 2013 @@ -369,8 +369,9 @@ this.progressMgr.updateAddressBook(this.gURL); if (Components.isSuccessCode(status) && data - && (data.toLowerCase().indexOf("begin:vcard") == 0)) - this.importCard(key, data); + && (data.toLowerCase().indexOf("begin:vcard") >= 0)) + // strip content (i.e. header lines) before "BEGIN:VCARD", + this.importCard(key, data.substring(data.toLowerCase().indexOf("begin:vcard"))); else this.appendFailure(status, key); @@ -385,8 +386,9 @@ this.progressMgr.updateAddressBook(this.gURL); if (Components.isSuccessCode(status) && data - && (data.toLowerCase().indexOf("begin:vlist") == 0)) - this.importList(key, data); + && (data.toLowerCase().indexOf("begin:vlist") >= 0)) + // strip content (i.e. header lines) before "BEGIN:VCARD", + this.importList(key, data.substring(data.toLowerCase().indexOf("begin:vlist"))); else this.appendFailure(status, key); if (this.remainingDownloads == 0) { |
|
Is there any chance to get this and the patch in Bug 1411 integrated to the next version of SOGo Connector? |
|
Hello Sven, we want to use Kerrio Connect 8.4 and Thunderbird (24.8.1) with the SOGo Connector (24.0.7). I found your diff and patched it, but it don´t work (debug thunderbird log: server 'https://192.168.10.209/carddav/users/domain.de/agattig/' is not a valid groupdav collection***) Hope you have a solution. Many thanks |
|
It was hard to find the correct URL to sync against Kerio, but mine is working for months now: https://<server>/contacts/<domain>.de/<user>/addressbook/ |
|
Many many thanks Sven. It works!!! :) have you a correct url for the global public adressbook? Original from Kerio KB: For public contacts |
|
I found the following working in the past: But you will have MANY contacts in there, since it lists all public contacts that are allowed to read in one directory. I cannot tell, where new contacts will be written to, but it might be worth trying... I personally only write to my personal addressbook and use the public as read-only LDAP (Base DN: fn=ContactRoot ) |
|
Big Thank you Sven, you solved all my problems. i tested the last 2 days and i got only errors. but your solution is perfect and it works! Quick Test with the public-contacts - it do all what we want :) specific public addressbook we don´t use at the moment. I wish you a happy Christmas. |
|
Fixed in commit 5c312ac5 |
|
Date Modified | Username | Field | Change |
---|---|---|---|
2013-12-13 08:55 | Sven Giermann | New Issue | |
2013-12-13 08:55 | Sven Giermann | File Added: skip_headers.diff | |
2013-12-13 08:56 | Sven Giermann | Note Added: 0006358 | |
2014-12-17 14:34 | agattig | Note Added: 0007864 | |
2014-12-18 08:06 | Sven Giermann | Note Added: 0007872 | |
2014-12-18 08:53 | agattig | Note Added: 0007873 | |
2014-12-18 10:50 | Sven Giermann | Note Added: 0007877 | |
2014-12-18 14:35 | agattig | Note Added: 0007881 | |
2016-07-12 12:05 | ludovic | Status | new => assigned |
2016-07-12 12:05 | ludovic | Assigned To | => ludovic |
2016-07-21 20:11 | ludovic | Note Added: 0010514 | |
2016-07-21 20:11 | ludovic | Status | assigned => closed |
2016-07-21 20:11 | ludovic | Resolution | open => suspended |