bcrypt.js
bcrypt.js copied to clipboard
Support bundling bcryptjs for client side usage
Right now, in order to use bcryptjs on browsers, we have to manually include the pre-bundled script into the main html file then access it via dcodeIO.bcrypt
. I think that it will be nicer if we can use bcryptjs like this instead.
import bcryptjs from 'bcryptjs';
bcryptjs.hash(...)
Then we can easily bundle it along with other js files with tools like webpack or rollup.
@lewispham can you provide a PR? Then @dcodeIO can review it and merge.
@Ruffio I think I can probably handle this but I'm just not sure about @dcodeIO 's opinion.
@lewispham - I'd say go for it anyway. Worst that happens is you end up with a fork that has a useful extra feature... I'm currently trying to get this to run under react native, and having no luck at all.
@lewispham would you provide a PR?
@jh3141 still have nothing so far for react native?
Sorry for the late reply. I've been a bit busy during the time. I've just created the PR #87 for this issue. Basically, this PR will add supports for bundling bcrypt.js via build tools like webpack
or rollup
. Please be aware that prebuilt dists and the global entry dcodeIO.bcrypt
will also be removed. You can feel free to give feedback here. Thanks.