search

denodrivers / mongo

🍃 MongoDB driver for Deno 🦕
https://deno.land/x/mongo
MIT License
508 stars 95 forks source link

Unnecessary dependencies #276

Closed jtoppine closed 2 years ago

jtoppine commented 2 years ago

Is there any chance the crypto-related dependencies could be refactored to use WebCrypto APIs, now that they are implemented and built-in in Deno?

At the moment, the mongo driver brings a whopping 1.76Mb of dependencies which alone makes for quite slow startup times for Deno apps. Most of the baggage seems to originate from crypto-related dependencies (especially pbkdf2). I believe moving to built-in crypto would bring the package to a fraction of current size.

erfanium commented 2 years ago

Last time I didn't find any native implementation for pbkdf2. Can you check it again?

jtoppine commented 2 years ago

Last time I didn't find any native implementation for pbkdf2. Can you check it again?

I'm not really familiar with the crypto API's, so I apologize if I'm missing something, but if you haven't checked in a while, the WebCrypto has been getting regular updates, and much of the pbkdf2 things appear to be done according to the roadmap issue here: https://github.com/denoland/deno/issues/11690