Attachment 114929 Details for Bug 89927 – Patch to handle google calendar error 403.

[patch] Patch to handle google calendar error 403.

0001-Handle-error-403-from-google-calendar-to-avoid-sync-.patch (text/plain), 1.74 KB, created by renato filho on 2015-04-07 17:24:58 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: renato filho

Created: 2015-04-07 17:24:58 UTC

Size: 1.74 KB

patch

obsolete

>From 4de0a0602ce6e8ff19b07fe09af66a385ea6dbf7 Mon Sep 17 00:00:00 2001
>From: Renato Araujo Oliveira Filho <renato.filho@canonical.com>
>Date: Tue, 7 Apr 2015 11:10:23 -0300
>Subject: [PATCH] Handle error 403 from google calendar to avoid sync to fail.
>
>---
> src/backends/webdav/WebDAVSource.cpp | 11 ++++++++++-
> 1 file changed, 10 insertions(+), 1 deletion(-)
>
>diff --git a/src/backends/webdav/WebDAVSource.cpp b/src/backends/webdav/WebDAVSource.cpp
>index 9f90cf4..9282ad8 100644
>--- a/src/backends/webdav/WebDAVSource.cpp
>+++ b/src/backends/webdav/WebDAVSource.cpp
>@@ -2251,7 +2251,11 @@ TrackingSyncSource::InsertItemResult WebDAVSource::insertItem(const string &uid,
>         // - Is retried? Might need slow sync in this case!
>         //
>         // req.addHeader("If-Match", etag);
>-        if (!req.run()) {
>+        std::set<int> expected;
>+        if (m_contextSettings->googleUpdateHack()) {
>+            expected = boost::assign::list_of(403);
>+        }
>+        if (!req.run(&expected)) {
>             goto retry;
>         }
>         SE_LOG_DEBUG(NULL, "update item status: %s",
>@@ -2265,6 +2269,11 @@ TrackingSyncSource::InsertItemResult WebDAVSource::insertItem(const string &uid,
>             // even when updating an item. Accept it.
>             // SE_THROW("unexpected creation instead of update");
>             break;
>+        case 403:
>+            // this can happen with Google Calendar if the user was removed from
>+            // an event by the organizer. Check bug #89927 for more details
>+            return InsertItemResult(new_uid, "", ITEM_OKAY);
>+            break;
>         default:
>             SE_THROW_EXCEPTION_STATUS(TransportStatusException,
>                                       std::string("unexpected status for update: ") +
>-- 
>2.1.0
>

Actions: View | Splinter Review

Attachments on bug 89927: 114898 | 114899 | 114929

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.