fix: fix sse connection logging

Author: kettanaito

src/browser/sse.ts

@@ -100,6 +100,23 @@ class ServerSentEventHandler<
 
     return super.predicate(args)
   }
+
+  async log(args: { request: Request; response: Response }): Promise<void> {
+    super.log({
+      request: args.request,
+
+      /**
+       * @note Construct a placeholder response since SSE response
+       * is being streamed and cannot be cloned/consumed at this point.
+       * This also allows us to rely on the same logging logic as in HTTP handlers.
+       */
+      response: new Response('[streaming]', {
+        status: args.response.status,
+        statusText: args.response.statusText,
+        headers: args.response.headers,
+      }),
+    })
+  }
 }
 
 type Values<T> = T[keyof T]

test/browser/sse-api/sse.server.connect.test.ts

@@ -41,6 +41,7 @@ test('makes the actual request when called "server.connect()"', async ({
 
         return new Response(stream, {
           headers: {
+            'access-control-allow-origin': '*',
             'content-type': 'text/event-stream',
             'cache-control': 'no-cache',
             connection: 'keep-alive',
@@ -74,7 +75,7 @@ test('makes the actual request when called "server.connect()"', async ({
   await expect(openPromise).resolves.toBeUndefined()
 })
 
-test('forwards message event from the server to the client automatically', async ({
+test.only('forwards message event from the server to the client automatically', async ({
   loadExample,
   page,
 }) => {
@@ -127,6 +128,8 @@ test('forwards message event from the server to the client automatically', async
     })
   }, url)
 
+  await page.pause()
+
   expect(message).toEqual({ message: 'hello' })
 })