moleculer-db icon indicating copy to clipboard operation
moleculer-db copied to clipboard

[WIP] add mongoose 6 support

Open devalexandre opened this issue 3 years ago • 4 comments

Adding support for mongoose 6.

however found some problems to maintain compatibility with version 5

TODO:

  • [ ] Error insertOne
[2022-04-06T01:23:18.765Z] INFO  melissa-403360/POSTS: MongoDB adapter has connected successfully.
[2022-04-06T01:23:18.771Z] INFO  melissa-403360/REGISTRY: 'posts' service is registered.
[2022-04-06T01:23:18.771Z] INFO  melissa-403360/POSTS: Service 'posts' started.
[2022-04-06T01:23:18.773Z] INFO  melissa-403360/BROKER: ✔ ServiceBroker with 2 service(s) started successfully in 37ms.
ERROR MongooseError: Operation `posts.insertOne()` buffering timed out after 10000ms
    at Timeout.<anonymous> (/home/alexandre/opensource/moleculer-mongo-6/node_modules/mongoose/lib/drivers/node-mongodb-native/collection.js:151:23)
    at listOnTimeout (internal/timers.js:557:17)
    at processTimers (internal/timers.js:500:7)

devalexandre avatar Apr 06 '22 01:04 devalexandre

You don't need to cover backward compatibility. We will release only the mongoose adapter as a new major version.

icebob avatar Apr 06 '22 18:04 icebob

ok, I will implement only version 6

devalexandre avatar Apr 06 '22 23:04 devalexandre

@devalexandre hello! Do you plan any updates on this PR?

Idris0v avatar May 31 '22 10:05 Idris0v

@devalexandre hello! Do you plan any updates on this PR?

Hi @Idris0v , I'm doing some tests

devalexandre avatar Jun 13 '22 15:06 devalexandre

@Idris0v finished :)

devalexandre avatar Sep 04 '22 03:09 devalexandre

@icebob mongoose 6 hasn't support node 10.

devalexandre avatar Sep 04 '22 03:09 devalexandre

your force-push killed my commits which fixed the Node 10 issue. Please revert it, or do it again.

icebob avatar Sep 11 '22 18:09 icebob

your force-push killed my commits which fixed the Node 10 issue. Please revert it, or do it again.

Okay, I'll close it and do it again.

devalexandre avatar Sep 11 '22 21:09 devalexandre