github/overleaf
1
0
Fork 0
mirror of https://github.com/overleaf/overleaf.git synced 2025-01-27 09:42:00 +00:00
Code Issues Projects Releases Packages Wiki Activity

Merge pull request #10376 from overleaf/jpa-pdf-caching-disable-first-error

Browse source 
[web] pdf-caching: disable pdf-caching after the first non-404 error

GitOrigin-RevId: f022e75acd9d5140ae02041b60af9819baa25e62
This commit is contained in:
Jakob Ackermann 2022-11-08 14:40:51 +00:00 committed by Copybot
parent aeade59677
commit f803563673
1 changed files with 7 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
services/web/frontend/js/features/pdf-preview/util/pdf-caching-transport.js
View file

@ -111,6 +111,13 @@ export function generatePdfCachingTransportFactory(PDFJS) {
.catch(err => {
if (abortSignal.aborted) return
if (isExpectedError(err)) {
if (is404(err)) {
// A regular pdf-js request would have seen this 404 as well.
} else {
// Flaky network, switch back to regular pdf-js requests.
metrics.failedCount++
metrics.failedOnce = true
}
throw new PDFJS.MissingPDFException()
}
metrics.failedCount++