@livechat/accounts-sdk 2.1.0 → 2.1.2

package/dist/accounts-sdk.js

@@ -3362,23 +3362,41 @@
    */
 
   function string(length) {
-    const bytes = new Uint8Array(length);
-    const result = [];
     const charset = '0123456789ABCDEFGHIJKLMNOPQRSTUVXYZabcdefghijklmnopqrstuvwxyz-._~';
+    const charsetLength = charset.length;
     const cryptoObj = window.crypto || window.msCrypto;
-    let random = '';
-    if (!cryptoObj) {
-      for (let i = 0; i < length; i++) {
-        random += charset.charAt(Math.floor(Math.random() * charset.length));
-      }
-    } else {
-      random = cryptoObj.getRandomValues(bytes);
+    const hasCrypto = cryptoObj && typeof cryptoObj.getRandomValues === 'function';
+    if (!hasCrypto) {
+      return generateWithMathRandom(length, charset, charsetLength);
     }
-    for (let a = 0; a < random.length; a++) {
-      result.push(charset[random[a] % charset.length]);
+    const maxByte = Math.floor(256 / charsetLength) * charsetLength;
+    if (maxByte === 0) {
+      return generateWithMathRandom(length, charset, charsetLength);
+    }
+    const result = [];
+    while (result.length < length) {
+      const remaining = length - result.length;
+      const buffer = new Uint8Array(remaining);
+      cryptoObj.getRandomValues(buffer);
+      for (let i = 0; i < buffer.length && result.length < length; i++) {
+        const value = buffer[i];
+        // Skip values that would cause modulo bias.
+        if (value >= maxByte) {
+          continue;
+        }
+        result.push(charset.charAt(value % charsetLength));
+      }
     }
     return result.join('');
   }
+  function generateWithMathRandom(length, charset, charsetLength) {
+    let output = '';
+    for (let i = 0; i < length; i++) {
+      const index = Math.floor(Math.random() * charsetLength);
+      output += charset.charAt(index);
+    }
+    return output;
+  }
   var random = {
     string: string
   };