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.

Sign up for GitHub

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

Jump to bottom

Replace CryptoJS by SJCL #5

Open

sembrestels opened this issue Dec 12, 2013 · 1 comment

sembrestels commented Dec 12, 2013

CryptoJS seems discontinued, while SJCL is being maintained very hard.

SJCL includes hmac and sha512 needed for the HD wallets, and it also has support for window.crypto.getRandomValues() (https://github.com/bitwiseshiftleft/sjcl/blob/master/core/random.js#L455).

I think SJCL is a better choice.

The text was updated successfully, but these errors were encountered:

Owner

vbuterin commented Dec 12, 2013

Interesting idea. I'll seriously consider it. Thanks a lot,

Vitalik

On 12/12/2013 01:55 AM, Sem wrote:

CryptoJS seems discontinued, while SJCL is being maintained very hard.

SJCL includes hmac and sha512 needed for the HD wallets, and it also
has support for |window.crypto.getRandomValues()|
(https://github.com/bitwiseshiftleft/sjcl/blob/master/core/random.js#L455).

I think SJCL is a better choice.

—
Reply to this email directly or view it on GitHub
#5.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment