Merge pull request #291 from jstastny/js/fix-utf8-size

Correctly calculate byte size of the utf8 strings
perry-mitchell · May 16, 2022 · cb9e9c1 · cb9e9c1
2 parents 1f9316f + 529c34f
commit cb9e9c1
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/source/tools/size.ts b/source/tools/size.ts
@@ -9,7 +9,7 @@ export function calculateDataLength(data: string | BufferLike): number {
     } else if (isBuffer(data)) {
         return (<Buffer>data).length;
     } else if (typeof data === "string") {
-        return (<string>data).length;
+        return Buffer.byteLength(<string>data);
     }
     throw new Layerr(
         {

diff --git a/test/node/tools/size.spec.js b/test/node/tools/size.spec.js
@@ -0,0 +1,8 @@
+const { calculateDataLength } = require("../../../dist/node/tools/size.js");
+
+describe("calculateDataLength", () => {
+    it("Correctly calculates length for utf-8 string", () => {
+        const utf8String = "řeřicha";
+        expect(calculateDataLength(utf8String)).to.equal(9);
+    });
+});