Skip to content
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.

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

Change math.random #1

Closed
0xFillin opened this issue Oct 15, 2021 · 1 comment
Closed

Change math.random #1

0xFillin opened this issue Oct 15, 2021 · 1 comment

Comments

@0xFillin
Copy link

I liked your brute force, but I'm not good at javascript and node js
Is it possible to screw the Web Crypto API here?

We tried to restore a wallet that was written out of order, but scattered, and I think if all the same we take the crypto.getRandomValues () method then it might work
https://developer.mozilla.org/en-US/docs/Web/API/Crypto/getRandomValues

Math.random() does not provide cryptographically secure random numbers. Do not use them for anything related to security. Use the Web Crypto API instead, and more precisely the window.crypto.getRandomValues() method.

@cnrad
Copy link
Owner

cnrad commented Oct 15, 2021

I appreciate the suggestion but I don't think it's necessary for this project. For this, math.random's purpose is to generate a random number between 0 and 2048 just to pick that word from a list, there's no need for it to be cryptographically secure because this project doesn't exactly deal with protective security, thanks again though

@cnrad cnrad closed this as completed Oct 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants