test: fix security warning
Add a simple implementation of getRandomValues to the browser context to get rid of the following warning: [SECURITY] node-uuid: crypto not usable, falling back to insecure Math.random()
This commit is contained in:
parent
58d9322190
commit
5f5e86e47f
|
@ -28,6 +28,17 @@ function createContext() {
|
||||||
// used by crypto-browserify & friends
|
// used by crypto-browserify & friends
|
||||||
Int32Array: Int32Array,
|
Int32Array: Int32Array,
|
||||||
DataView: DataView,
|
DataView: DataView,
|
||||||
|
crypto: {
|
||||||
|
getRandomValues: function(typedArray) {
|
||||||
|
var randomBuffer = require('crypto').randomBytes(typedArray.length);
|
||||||
|
// This implementation is not secure: we take random 8bit values
|
||||||
|
// and assign them to 8/16/32bit values, leaving high-order bits
|
||||||
|
// filled with zeroes.
|
||||||
|
// Fortunately, the bootstrapping process does not rely on secure
|
||||||
|
// random numbers, therefore we can afford such shortcut.
|
||||||
|
typedArray.set(randomBuffer);
|
||||||
|
},
|
||||||
|
},
|
||||||
|
|
||||||
// allow the browserified code to log messages
|
// allow the browserified code to log messages
|
||||||
// call `printContextLogs(context)` to print the accumulated messages
|
// call `printContextLogs(context)` to print the accumulated messages
|
||||||
|
|
Loading…
Reference in New Issue