Skip to content

Use Cursor.next instead of Cursor.nextObject#66

Open
aslafy-z wants to merge 1 commit intocayasso:masterfrom
aestetype:fix/upgrade-mongo
Open

Use Cursor.next instead of Cursor.nextObject#66
aslafy-z wants to merge 1 commit intocayasso:masterfrom
aestetype:fix/upgrade-mongo

Conversation

@aslafy-z
Copy link
Copy Markdown

Breaking change in mongodb package prevent passing an mongodb instance from version >=3.0.

Cursor.nextObject is deprecated since 2.0 and removed from api since 3.0

Similar: aheckmann/gridfs-stream#125

@aslafy-z aslafy-z force-pushed the fix/upgrade-mongo branch from 69a927a to 700a5b6 Compare June 21, 2018 15:40
@boutell
Copy link
Copy Markdown

boutell commented Aug 25, 2018

Hmm, is this module still being actively maintained? We just went through the 3.x compatibility process for other things, so this is a little worrisome.

@boutell
Copy link
Copy Markdown

boutell commented Aug 27, 2018

Thanks for the workaround, but my concern was more about long term maintenance of the module etc. In any case I ultimately decided that given the way MongoDB hosting is priced relative to S3 and relatives, I can't really justify keeping files in it due to their size, even if it's one less separate cost and separate service to manage. But maybe that will be worth it to some client at some point...

@cayasso
Copy link
Copy Markdown
Owner

cayasso commented Aug 27, 2018

Hi guys, thanks for contributing to the project, I will review this tomorrow morning, I am jam with bunch of work.

@aslafy-z
Copy link
Copy Markdown
Author

@cayasso Could you please include this fix in the next release ? Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants