Skip to content
This repository has been archived by the owner on Aug 12, 2022. It is now read-only.

Node 6.0+ support is broken #51

Open
nullptr128 opened this issue Jun 15, 2016 · 6 comments
Open

Node 6.0+ support is broken #51

nullptr128 opened this issue Jun 15, 2016 · 6 comments

Comments

@nullptr128
Copy link

nullptr128 commented Jun 15, 2016

Hello!

Promised-Mongo unfortunately does not work on Node 6.0 at all, while it works correctly in Node 4.X.

It keeps crashing on /promised-mongo/dist/Cursor.js:65

case 3:
            this._cursor = context$2$0.sent.cursor(this.namespace, this.command, this.options);

Saying that send.cursor is not a function. I am not sure if it is only your file or whole regenerator runtime is broken on 6.0 but as far as I know, you can use babel-plugin-transform-async-to-generator instead as generator methods are supported on 6.0.

@Ice32
Copy link

Ice32 commented Aug 24, 2016

Having the same issue here, it works on Node 5.0.0, but I'm getting "context$2$0.sent.cursor is not a function" on 6.0.0

@Wolke
Copy link

Wolke commented Sep 16, 2016

same issue

@kpdecker
Copy link

Root issue here is that _serverPromise is not set, which breaks the proxy generator here:
https://github.com/gordonmleigh/promised-mongo/blob/master/index.js#L28

Confirmed that the cursor issue above is fixed when changing that to remove _serverPromise from the collection generation logic, but this feels like whackamole, so I'm not sure what the ideal fix would be here.

@Darkhogg
Copy link

Darkhogg commented Nov 1, 2016

That is not the only issue, however. As soon as I try to connect I get various errors about not being able to add _id to a string, or about Symbols not being strings, etc.

This is a serious issue!

@ivan133
Copy link

ivan133 commented Oct 2, 2017

So, it shouldn't work with nove 6+, right?

@Dussed
Copy link

Dussed commented Oct 3, 2017

Had a similar issue using Node 6.11.0 - I've found dropping then-mongo in place works for me, with a few minor changes (pretty much just imports) - it's also based off of mongojs

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants