refactor: use lowercase response with HTTP long-polling

In order to match the behavior of the Node.js implementation. Reference: https://github.com/socketio/engine.io/blob/917d1d29e13f2e8f523c3738f6413f67b587aebe/lib/transports/polling.ts#L174
socketio · Sep 14, 2022 · c8c48a4 · c8c48a4
1 parent 3b1eb82
commit c8c48a4
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/packages/engine.io/lib/transports/polling.ts b/packages/engine.io/lib/transports/polling.ts
@@ -104,7 +104,7 @@ export class Polling extends Transport {
     }
 
     return Promise.resolve(
-      new Response("OK", {
+      new Response("ok", {
         status: 200,
         headers: responseHeaders,
       }),

diff --git a/packages/engine.io/test/close.test.ts b/packages/engine.io/test/close.test.ts
@@ -174,7 +174,7 @@ describe("close", () => {
       assertEquals(await pollResponse.text(), "6");
 
       assertEquals(dataResponse.status, 200);
-      assertEquals(await dataResponse.text(), "OK");
+      assertEquals(await dataResponse.text(), "ok");
 
       partialDone();
     });
@@ -262,7 +262,7 @@ describe("close", () => {
 
       const body = await dataResponse.text();
 
-      assertEquals(body, "OK");
+      assertEquals(body, "ok");
 
       partialDone();
     });