Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use Cursor.next instead of Cursor.nextObject #66

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aslafy-z
Copy link

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

@boutell
Copy link

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

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
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
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