fix: convert to ascii before thirty-two encode

.github/workflows/main.yml

@@ -25,6 +25,7 @@ jobs:
       run: |
         npm run setup
         npm run test
+        npm run test:extras -- --type=issue
       env:
         PUPPETEER_SKIP_CHROMIUM_DOWNLOAD: true
 

.prettierignore

@@ -1,7 +1,6 @@
 .cache
 builds
 externals
-extras
 node_modules
 package-lock.json
 public

README.md

@@ -50,7 +50,7 @@
 It implements both [HOTP][rfc-4226-wiki] - [RFC 4226][rfc-4226]
 and [TOTP][rfc-6238-wiki] - [RFC 6238][rfc-6238],
 and are tested against the test vectors provided in their respective RFC specifications.
-These datasets can be found in the `packages/tests-data` folder.
+These datasets can be found in the `tests/data` folder.
 
 - [RFC 4226 Dataset][rfc-4226-dataset]
 - [RFC 6238 Dataset][rfc-6238-dataset]

packages/otplib-plugin-thirty-two/src/index.ts

@@ -27,7 +27,7 @@ export const keyEncoder: KeyEncoder = (
   encoding: KeyEncodings
 ): Base32SecretKey => {
   return thirtyTwo
-    .encode(Buffer.from(secret, encoding).toString())
+    .encode(Buffer.from(secret, encoding).toString('ascii'))
     .toString()
     .replace(/=/g, '');
 };

tests/extras/issue-201.test.js

@@ -0,0 +1,28 @@
+import { authenticatorGenerateSecret, KeyEncodings } from '@otplib/core';
+import { createRandomBytes } from '@otplib/plugin-crypto';
+import * as thirtyTwo from '@otplib/plugin-thirty-two';
+import * as base32EncDec from '@otplib/plugin-base32-enc-dec';
+
+[
+  {
+    name: 'thirty-two',
+    mod: thirtyTwo
+  },
+  {
+    name: 'base32-enc-dec',
+    mod: base32EncDec
+  }
+].forEach(({ name, mod }) => {
+  const options = {
+    createRandomBytes,
+    encoding: KeyEncodings.HEX,
+    keyEncoder: mod.keyEncoder,
+    keyDecoder: mod.keyDecoder
+  };
+
+  test(`[${name}] generated secret should have consistent length`, async () => {
+    for (let i = 0; i < 20; i++) {
+      expect(authenticatorGenerateSecret(10, options)).toHaveLength(16);
+    }
+  });
+});