From a64c6808e5208865a3d7f53c2f3b5ba4d914c027 Mon Sep 17 00:00:00 2001 From: Dan Stillman Date: Fri, 7 Apr 2017 22:34:19 -0400 Subject: [PATCH] Always include server response in Zotero.HTTP.request() errors --- chrome/content/zotero/xpcom/http.js | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/chrome/content/zotero/xpcom/http.js b/chrome/content/zotero/xpcom/http.js index 3fc2521a8..3392de2f0 100644 --- a/chrome/content/zotero/xpcom/http.js +++ b/chrome/content/zotero/xpcom/http.js @@ -252,10 +252,8 @@ Zotero.HTTP = new function() { } deferred.resolve(xmlhttp); } else { - let msg = "HTTP " + method + " " + dispURL + " failed with status code " + xmlhttp.status; - if (xmlhttp.status == 400 || options.debug) { - msg += ":\n\n" + xmlhttp.responseText; - } + let msg = "HTTP " + method + " " + dispURL + " failed with status code " + xmlhttp.status + + ":\n\n" + xmlhttp.responseText; Zotero.debug(msg, 1); deferred.reject(new Zotero.HTTP.UnexpectedStatusException(xmlhttp, msg)); }