https://bugs.freedesktop.org/show_bug.cgi?id=89853
Patrick Ohly <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO --- Comment #4 from Patrick Ohly <[email protected]> --- (In reply to renato filho from comment #3) > works nice with syncevolution from trunk. One the same device, using the same settings? There is one relevant change (see below), but it is hard to tell why it should avoid the issue, or what the issue really is. Because of the low loglevel, the actual request was not in the log. Better run with loglevel=4. commit 75115022c525d21341be4113d789eccd6e23cca8 Author: Patrick Ohly <[email protected]> Date: Thu Feb 12 13:36:42 2015 +0100 CalDAV: more efficient "is empty" check (FDO #86335) Since 1.4.99.4, syncing WebDAV collections always checks first whether there are items in the collections. This was partly done for slow sync prevention (which is not necessary for empty collections), partly for the "is the datastore usable" check. However, this did not take into account that for CalDAV collections, the entire content gets downloaded for this check. That is because filtering by item type (VEVENT vs. VJOURNAL) is not implemented correctly by all servers. So now all CalDAV syncs, whether incremental or slow, always transfered all items, which is not the intention (incremental syncs should be fast and efficient). This commit adds a more efficient isEmpty() check: for simple CardDAV collections, only luid and etag get transferred, as in listAllItems(). This is the behavior from 1.5. For CalDAV, a report with a filter for the content type is used and the transfer gets aborted after the first item, without actually double-checking the content of the item. This is different from listAllItems(), which really transfers the content. This extra content check would only be needed for some old servers (Radical 0.7) and is not essential, because reporting "not empty" even when empty is safe. commit 7e5638fd904a393d5cc344c075857e3695c7fb80 Author: Patrick Ohly <[email protected]> Date: Thu Feb 12 13:14:59 2015 +0100 WebDAV: utility class for ne_status Besides offering C++ wrappers for the C API, this class also ensures that memory for the "reason_string" is properly freed. commit 85df49a68b82000f1429c577fb41cc4259359ee5 Author: Patrick Ohly <[email protected]> Date: Thu Feb 12 13:12:16 2015 +0100 WebDAV: enhance report handling (status, aborting) Capture the item status and pass it to the response handler. Response handlers are allowed to return a non-zero integer when using the initAbortingReportParser(), which then aborts processing of the response. This leads to errors being returned by ne_xml_dispatch_request(). Session::run() needs to be told if the request was aborted, in which case all errors are ignored. -- You are receiving this mail because: You are on the CC list for the bug. You are the assignee for the bug.
_______________________________________________ Syncevolution-issues mailing list [email protected] https://lists.syncevolution.org/mailman/listinfo/syncevolution-issues
