|
|
|
|
|
by Macha
1990 days ago
|
|
Our security team at a previous employer previously added a systemwide checker to our github enterprise installation that would spam comments on any change to a file in which Math.random is used. The idea is that anyone using random numbers must be implementing a cryptographic protocol and therefore should not be using Math.random as it's not a CSPRNG. So all the AB tests, percentage rollouts etc. started getting spam PR comments until they were made to turn it back off again. Frankly if a teammate was writing their own crypto algorithm implemntation in the bog standard web app we working on, that would be more concerning than which RNG they're using. |
|