Merge pull request #17843 from overleaf/em-fix-422-resyncs

Change in history 422 error messages

GitOrigin-RevId: 414642c838703c9084058e1204192ec1cbb1e0a3

services/project-history/app/js/ErrorRecorder.js

@@ -137,6 +137,10 @@ async function getFailures() {
     'Error: bad response from filestore: 404': 'filestore-404',
     'Error: bad response from filestore: 500': 'filestore-500',
     'NotFoundError: got a 404 from web api': 'web-api-404',
+    'OError: history store a non-success status code: 413': 'history-store-413',
+    'OError: history store a non-success status code: 422': 'history-store-422',
+    'OError: history store a non-success status code: 500': 'history-store-500',
+    'OError: history store a non-success status code: 503': 'history-store-503',
     'Error: history store a non-success status code: 413': 'history-store-413',
     'Error: history store a non-success status code: 422': 'history-store-422',
     'Error: history store a non-success status code: 500': 'history-store-500',

services/project-history/app/js/RetryManager.js

@@ -20,6 +20,7 @@ const TEMPORARY_FAILURES = [
 
 const HARD_FAILURES = [
   'Error: history store a non-success status code: 422',
+  'OError: history store a non-success status code: 422',
   'OpsOutOfOrderError: project structure version out of order',
   'OpsOutOfOrderError: project structure version out of order on incoming updates',
   'OpsOutOfOrderError: doc version out of order',

services/project-history/scripts/clear_deleted_history.js

@@ -143,7 +143,7 @@ function checkAndClear(project, callback) {
 // find all the broken projects from the failure records
 async function main() {
   const results = await db.projectHistoryFailures
-    .find({ error: 'Error: history store a non-success status code: 422' })
+    .find({ error: /history store a non-success status code: 422/ })
     .toArray()
 
   console.log('number of queues without history store 442 =', results.length)

services/project-history/scripts/force_resync.js

@@ -198,6 +198,7 @@ function checkAndClear(project, callback) {
 // find all the broken projects from the failure records
 const errorsToResync = [
   'Error: history store a non-success status code: 422',
+  'OError: history store a non-success status code: 422',
   'OpsOutOfOrderError: project structure version out of order',
 ]