Merge pull request #63 from micaksica2/master

Make randomDigits use crypto.randomBytes
feathersjs-ecosystem · Aug 6, 2017 · 1aafcac · 1aafcac
2 parents efa4d80 + 19cfc32
commit 1aafcac
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/src/helpers.js b/src/helpers.js
@@ -39,8 +39,13 @@ const randomBytes = len => new Promise((resolve, reject) => {
 });
 
 const randomDigits = len => {
-  const str = Math.random().toString() + Array(len + 1).join('0');
-  return str.substr(2, len);
+  let str = '';
+
+  while (str.length < len) {
+    str += parseInt('0x' + crypto.randomBytes(4).toString('hex')).toString();
+  }
+
+  return str.substr(0, len);
 };
 
 const getLongToken = len => randomBytes(len);