Ver Fonte

Merge pull request #8546 from overleaf/bg-fix-docupdater-history-redis

[document-updater] docsWithHistoryOps should use the main rclient

GitOrigin-RevId: a86c081ea0b0f02a9d4c36df0ca76dc9aa3bac57
Brian Gough há 4 anos atrás
pai
commit
e7e4e49d90

+ 1 - 3
services/document-updater/app/js/HistoryRedisManager.js

@@ -12,9 +12,7 @@
  */
 let HistoryRedisManager
 const Settings = require('@overleaf/settings')
-const rclient = require('@overleaf/redis-wrapper').createClient(
-  Settings.redis.history
-)
+const { rclient } = require('./RedisManager') // docsWithHistoryOps lives in main redis
 const Keys = Settings.redis.history.key_schema
 const logger = require('@overleaf/logger')
 

+ 1 - 1
services/document-updater/test/unit/js/HistoryRedisManager/HistoryRedisManagerTests.js

@@ -25,7 +25,7 @@ describe('HistoryRedisManager', function () {
     this.rclient.multi = () => this.rclient
     this.HistoryRedisManager = SandboxedModule.require(modulePath, {
       requires: {
-        '@overleaf/redis-wrapper': { createClient: () => this.rclient },
+        './RedisManager': { rclient: this.rclient },
         '@overleaf/settings': {
           redis: {
             history: (this.settings = {