mirrors/zotero
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

Fix hang on 500 from site during Find Available PDF

Browse source 
Find Available PDF has its own domain-based retry logic that predated
automatic 5xx retries in Zotero.HTTP, so disable the latter and fix some
bugs in the former.

Fixes #2700
This commit is contained in:
Dan Stillman 2022-10-29 04:07:19 -04:00
parent 77fdc8e40c
commit 7f91a0c1a8
1 changed files with 8 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
chrome/content/zotero/xpcom/attachments.js
View file

@ -2085,7 +2085,9 @@ Zotero.Attachments = new function(){
nextURL,
{
responseType: 'blob',
followRedirects: false
followRedirects: false,
// Use our own error handling
errorDelayMax: 0
}
);
}
@ -2096,11 +2098,13 @@ Zotero.Attachments = new function(){
noDelay = false;
continue;
}
throw e;
}
break;
}
afterRequest(nextURL);
if (!req) {
break;
}
if ([301, 302, 303, 307].includes(req.status)) {
let location = req.getResponseHeader('Location');
if (!location) {
@ -2158,7 +2162,7 @@ Zotero.Attachments = new function(){
}
// If DOI resolves directly to a PDF, save it to disk
if (contentType.startsWith('application/pdf')) {
if (contentType && contentType.startsWith('application/pdf')) {
Zotero.debug("URL resolves directly to PDF");
await Zotero.File.putContentsAsync(path, blob);
await _enforcePDF(path);
@ -2174,7 +2178,7 @@ Zotero.Attachments = new function(){
continue;
}
if (!url) {
Zotero.debug(`No PDF found on ${responseURL}`);
Zotero.debug(`No PDF found on ${responseURL || pageURL}`);
continue;
}
if (isTriedURL(url)) {