fix: invalid character error in cookie serialize

contributors.yml

@@ -4,6 +4,7 @@
 - abotsi
 - ahbruns
 - ahmedeldessouki
+- aiji42
 - airjp73
 - airondumael
 - Alarid

packages/remix-server-runtime/__tests__/cookies-test.ts

@@ -52,6 +52,14 @@ describe("cookies", () => {
     expect(value).toMatchInlineSnapshot(`"hello michael"`);
   });
 
+  it("parses/serializes string values containing non-ascii characters", async () => {
+    let cookie = createCookie("my-cookie");
+    let setCookie = await cookie.serialize("日本語");
+    let value = await cookie.parse(getCookieFromSetCookie(setCookie));
+
+    expect(value).toBe("日本語");
+  });
+
   it("fails to parses signed string values with invalid signature", async () => {
     let cookie = createCookie("my-cookie", {
       secrets: ["secret1"],

packages/remix-server-runtime/cookies.ts

@@ -166,13 +166,29 @@ async function decodeCookieValue(
 }
 
 function encodeData(value: any): string {
-  return btoa(JSON.stringify(value));
+  return btoa(toBinary(JSON.stringify(value)));
 }
 
 function decodeData(value: string): any {
   try {
-    return JSON.parse(atob(value));
+    return JSON.parse(fromBinary(atob(value)));
   } catch (error) {
     return {};
   }
 }
+
+function toBinary(value: string): string {
+  const codeUnits = new Uint16Array(value.length);
+  for (let i = 0; i < codeUnits.length; i++) {
+    codeUnits[i] = value.charCodeAt(i);
+  }
+  return String.fromCharCode(...new Uint8Array(codeUnits.buffer));
+}
+
+function fromBinary(value: string): string {
+  const bytes = new Uint8Array(value.length);
+  for (let i = 0; i < bytes.length; i++) {
+    bytes[i] = value.charCodeAt(i);
+  }
+  return String.fromCharCode(...new Uint16Array(bytes.buffer));
+}