Support mongodb gridfs about keyv

[lukechilds]

Hi, sorry, I'm not too familiar with gridfs and what it is. Is this just an option in the driver?

If so you can do this by passing the option down to the driver through the storage adapter. You can see an example of how this is done with keyv-sqlite. You can see how the table and busyTimeout options are passed through there.

[adityapatadia]

It's different class in same driver http://mongodb.github.io/node-mongodb-native/3.1/api/GridFSBucket.html

I will implement it in my fork so you have reference implementation. It's very crucial feature for me so I am not creating PR for now as I need it very fast.

[adityapatadia]

I have published new package here: https://www.npmjs.com/package/keyv-mongo-gridfs You can refer to it anytime. Let me know if you need any help.

[thgh]

Hi @adityapatadia thanks for your improvements, do you consider making a pull request now?

You can also clarify how to use the adapter in your readme:

// How to use this adapter
import Keyv from 'keyv'
import MongoAdapter from 'keyv-mongo-gridfs'

const store = new MongoAdapter({
  url: process.env.MONGO_CONNECTION_STRING,
  db: 'some-db',
})
const keyv = new Keyv(connectionString, { store })

[adityapatadia]

I have stopped development of the same but that module was pretty stable. I am not able to create PR right now. May be after some I will do it.

I have updated the readme.

[jaredwray]

@adityapatadia - do you have the source code available for this as I was looking to see if we should add this in as a feature?

[adityapatadia]

Source code is here: https://github.com/gumlet/keyv-mongo-gridfs

[jaredwray]

@adityapatadia - I am considering rolling this into the main driver as an option and was wondering if you would be ok with that?

[adityapatadia]

I am perfectly fine as long as you acknowledge my contribution. Please feel free to take any code. — Regards, Aditya Patadia

…

On 21-Oct-2021, at 8:04 PM, Jared Wray ***@***.***> wrote:  @adityapatadia - I am considering rolling this into the main driver as an option and was wondering if you would be ok with that? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.

[jaredwray]

I am perfectly fine as long as you acknowledge my contribution. Please feel free to take any code. — Regards, Aditya Patadia
…
On 21-Oct-2021, at 8:04 PM, Jared Wray @.***> wrote:  @adityapatadia - I am considering rolling this into the main driver as an option and was wondering if you would be ok with that? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.

Will add you in the pull request when we do this! Thanks so much

[jaredwray]

@adityapatadia and @thgh - this is now part of the core MongoDB and will be fully live on NPM today.

@adityapatadia - Would you like us to just list the idea as a contributor as we ended up not using any of your code but want to recognize your work on this. Just let us know!

[adityapatadia]

As you wish and whatever works best. :) — Regards, Aditya Patadia

…

On 26-Jan-2022, at 11:55 PM, Jared Wray ***@***.***> wrote:  @adityapatadia and @thgh - this is now part of the core MongoDB and will be fully live on NPM today. @adityapatadia - Would you like us to just list the idea as a contributor as we ended up not using any of your code but want to recognize your work on this. Just let us know! — Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android. You are receiving this because you were mentioned.

[jaredwray]

Will do. Working on adding in contributions as a listing :-)