Skip to content
This repository has been archived by the owner on Jul 1, 2024. It is now read-only.

Review implementation of cryptographic random numbers #12

Open
chris-giblin opened this issue Apr 22, 2022 · 0 comments
Open

Review implementation of cryptographic random numbers #12

chris-giblin opened this issue Apr 22, 2022 · 0 comments
Assignees

Comments

@chris-giblin
Copy link
Contributor

Review, document, implement approach to generation of cryptographic random numbers.
Commenting on CryptoUtils.java, @tombentley points out in PR #9 :

Is there a reason to not use a static SecureRandom? Calling nextBytes() on a fresh instance means it will have to seed from an entropy source (I'm assuming /dev/random or /dev/urandom is used in some form or other) every time, which might end up blocking if there's not enough entropy.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants