Closed prestwich closed 5 years ago
Is this in reference to _get_random_challenge_string
?
I think so.
Yeah that's what brought it to my attention. I noticed a few other places where we could use non-reproducible randomness, like in Keyring
.
Okay, I think this was fixed in the last commit but I'll leave it open as a reminder to use os.urandom
in te future like in Keyring
as @frdwrd mentioned.
👋 Hey! Thanks for this contribution. Apologies for the delay in responding!
We've decided to rearchitect Storj, so that we can scale better. You can read more about this decision here. This means that we are entirely focused on v3 at the moment, in the storj/storj repository. Our white paper for v3 is coming very, very soon - follow along on the blog and in our Rocketchat.
As this repository is part of the v2 network, we're no longer maintaining this repository. I am going to close this for now. If you have any questions, I encourage you to jump on Rocketchat and ask them there. Thanks!
Best practices suggest
os.urandom
orrandom.SystemRandom