From cf8557b7f12721f75c91614f2ea1adeb302db732 Mon Sep 17 00:00:00 2001 From: Brian Gough Date: Thu, 12 Dec 2024 11:11:14 +0000 Subject: [PATCH] Merge pull request #22481 from overleaf/bg-fix-blob-redirect-middleware fix fileToBlobRedirectMiddleware internal redirect GitOrigin-RevId: 874196d58f8b657d73a8e7f1ea4fef8290d48a10 --- .../web/app/src/Features/History/HistoryController.js | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/services/web/app/src/Features/History/HistoryController.js b/services/web/app/src/Features/History/HistoryController.js index 19dfb3eb21..0380f51899 100644 --- a/services/web/app/src/Features/History/HistoryController.js +++ b/services/web/app/src/Features/History/HistoryController.js @@ -85,14 +85,16 @@ module.exports = HistoryController = { if (err) { return next(err) } - if (file?.hash) { - req.url = `/project/${projectId}/blob/${file.hash}` - } metrics.inc('fileToBlobRedirectMiddleware', 1, { method: req.method, status: Boolean(file?.hash), }) - next() + if (file?.hash) { + req.url = `/project/${projectId}/blob/${file.hash}` + next('route') // redirect to blob route + } else { + next() // continue with file route + } } ) },