Document is already bound to this RedisPersistence instance

[hanspagel]

• 2 instances connected with Redis

Error:
[2021-04-22T22:09:08.111Z] New connection to "new-doc" …
[2021-04-22T22:09:08.111Z] Created document "new-doc" …
(node:53740) UnhandledPromiseRejectionWarning: Error: "new-doc" is already bound to this RedisPersistence instance
    at Object.error.create (~/repo/hocuspocus/node_modules/lib0/error.js:12:28)
    at RedisPersistence.bindState (~/repo/hocuspocus/node_modules/y-redis/src/y-redis.js:158:13)
    at Redis.onCreateDocument (file://~/repo/hocuspocus/node_modules/@hocuspocus/extension-redis/src/Redis.ts:45:28)
    at file://~/repo/hocuspocus/node_modules/@hocuspocus/server/src/Hocuspocus.ts:288:74
...

Setup:
extensions: [
    // Log level is not configurable at the moment.
    new Logger(),
    new Redis({
      host: process.env.REDIS_HOST,
      port: process.env.REDIS_PORT,
    }),
    new RocksDB({
      // [required] Path to the directory to store the actual data in
      path: "./database",

      // [optional] Configuration options for theRocksDB adapter, defaults to "{}“
      options: {
      // This option is only a example. See here for a full list:
      // https://www.npmjs.com/package/leveldown#options
        createIfMissing: true,
      },
   }),
],

[jamesopti]

I debugged this locally and believe I have a simple solution.

This repo is not open for forks, so I manually cloned it and opened a PR to my clone. Happy to update the base of the PR if forks are enabled!

https://github.com/gamma-app/hocuspocus-ueberdosis/pull/1/files#diff-d77a87bababf87a73bdd805021f23aee4b71388e7ff5942d62626858c7bed76f

[hanspagel]

Sorry for the long silence. Good news though! We’ll start to work on a complete overhaul of the Redis extension for hocuspocus together with the creator of Y.js next week. ✌️

And thanks for the PR, I’ll take a look at it!

[bertpareyn]

We are currently setting up and running into this as well, any chance this could be merged back soon? 🙏

[hanspagel]

Thanks @jamesopti! I’ve incorporated your fix and released a new version. 🙌