cayasso / mongo-oplog

Watch mongodb oplog in a simple way
381 stars 91 forks source link

Use Cursor.next instead of Cursor.nextObject #66

Open aslafy-z opened 6 years ago

aslafy-z commented 6 years ago

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: https://github.com/aheckmann/gridfs-stream/issues/125

boutell commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

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

aslafy-z commented 6 years ago

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