From 8f742bc7abb0f68ce3e146922c89549d75d5cdc5 Mon Sep 17 00:00:00 2001 From: Dan Stillman Date: Tue, 8 Nov 2011 02:45:17 -0500 Subject: [PATCH] Don't require manual sync on 40x response for invalid session errors Server currently returns 500 but should return 403 --- chrome/content/zotero/xpcom/sync.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/chrome/content/zotero/xpcom/sync.js b/chrome/content/zotero/xpcom/sync.js index 87b4ecee3b..0956f4a997 100644 --- a/chrome/content/zotero/xpcom/sync.js +++ b/chrome/content/zotero/xpcom/sync.js @@ -1849,7 +1849,7 @@ Zotero.Sync.Server = new function () { if (firstChild.localName == 'error') { // Don't automatically retry 400 errors - if (xmlhttp.status >= 400 && xmlhttp.status < 500) { + if (xmlhttp.status >= 400 && xmlhttp.status < 500 && !_invalidSession(xmlhttp)) { Zotero.debug("Server returned " + xmlhttp.status + " -- manual sync required", 2); Zotero.Sync.Server.manualSyncRequired = true; }