From 258c6224f48984248107c62c67a05347a283ebba Mon Sep 17 00:00:00 2001 From: Andrew Rumble Date: Thu, 15 Jan 2026 10:31:32 +0000 Subject: [PATCH] Merge pull request #30675 from overleaf/ar-jpa-log-missing-history-id-docupdater [document-updater] add additional logging when doc is missing projectHistoryId GitOrigin-RevId: fba311cb10e0eeeb0a8165ce4e519c7704a7458f --- services/document-updater/app/js/PersistenceManager.js | 7 +++++++ services/document-updater/app/js/RedisManager.js | 7 +++++++ 2 files changed, 14 insertions(+) diff --git a/services/document-updater/app/js/PersistenceManager.js b/services/document-updater/app/js/PersistenceManager.js index 6e832f9aa7..c357812326 100644 --- a/services/document-updater/app/js/PersistenceManager.js +++ b/services/document-updater/app/js/PersistenceManager.js @@ -102,6 +102,13 @@ function getDoc(projectId, docId, options = {}, _callback) { body.ranges = {} } + if (!body.projectHistoryId) { + logger.warn( + { projectId, docId }, + 'projectHistoryId not found for doc from web' + ) + } + callback( null, body.lines, diff --git a/services/document-updater/app/js/RedisManager.js b/services/document-updater/app/js/RedisManager.js index 1dc20fdf38..84bc288872 100644 --- a/services/document-updater/app/js/RedisManager.js +++ b/services/document-updater/app/js/RedisManager.js @@ -282,6 +282,13 @@ const RedisManager = { }) } + if (!projectHistoryId) { + logger.warn( + { docId, projectId }, + 'projectHistoryId not found for doc in Redis' + ) + } + return { lines: docLines, version,