Преглед изворни кода

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
Andrew Rumble пре 9 месеци
родитељ
комит
258c6224f4

+ 7 - 0
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,

+ 7 - 0
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,