fix(middleware/cors): explicitly return No Content for statusText in …

…options request (#3719)
honojs · Dec 4, 2024 · fc8fb0f · fc8fb0f
1 parent 80c7e22
commit fc8fb0f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/middleware/cors/index.test.ts b/src/middleware/cors/index.test.ts
@@ -81,6 +81,7 @@ describe('CORS by Middleware', () => {
     const res = await app.request(req)
 
     expect(res.status).toBe(204)
+    expect(res.statusText).toBe('No Content')
     expect(res.headers.get('Access-Control-Allow-Methods')?.split(',')[0]).toBe('GET')
     expect(res.headers.get('Access-Control-Allow-Headers')?.split(',')).toEqual([
       'X-PINGOTHER',

diff --git a/src/middleware/cors/index.ts b/src/middleware/cors/index.ts
@@ -137,7 +137,7 @@ export const cors = (options?: CORSOptions): MiddlewareHandler => {
       return new Response(null, {
         headers: c.res.headers,
         status: 204,
-        statusText: c.res.statusText,
+        statusText: 'No Content',
       })
     }
     await next()