Merge pull request #250 from MasterKale/fix/optimize-convert-cert-buffer-to-pem

fix/optimize-convert-cert-buffer-to-pem

1 files changed, 7 insertions, 5 deletions

diff --git a/packages/server/src/helpers/convertCertBufferToPEM.ts b/packages/server/src/helpers/convertCertBufferToPEM.ts
index 8de140d..b6949c4 100644
--- a/packages/server/src/helpers/convertCertBufferToPEM.ts
+++ b/packages/server/src/helpers/convertCertBufferToPEM.ts
@@ -5,15 +5,17 @@ import type { Base64URLString } from '@simplewebauthn/typescript-types';
  * Convert buffer to an OpenSSL-compatible PEM text format.
  */
 export function convertCertBufferToPEM(certBuffer: Buffer | Base64URLString): string {
-  let buffer: Buffer;
+  let b64cert: string;
+
+  /**
+   * Get certBuffer to a base64 representation
+   */
   if (typeof certBuffer === 'string') {
-    buffer = base64url.toBuffer(certBuffer);
+    b64cert = base64url.toBase64(certBuffer);
   } else {
-    buffer = certBuffer;
+    b64cert = certBuffer.toString('base64');
   }
 
-  const b64cert = buffer.toString('base64');
-
   let PEMKey = '';
   for (let i = 0; i < Math.ceil(b64cert.length / 64); i += 1) {
     const start = 64 * i;