fix: header sent but not destroyed should be destroyed

honojs · Nov 17, 2023 · 72cdce6 · 72cdce6
1 parent e1625e6
commit 72cdce6
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/src/listener.ts b/src/listener.ts
@@ -79,11 +79,17 @@ export const getRequestListener = (fetchCallback: FetchCallback) => {
     }
 
     // do not write response if outgoing is already finished
-    if (outgoing.destroyed || outgoing.writableEnded || outgoing.headersSent) {
+    if (outgoing.destroyed || outgoing.writableEnded) {
       console.info('The response is already finished.')
       return
     }
 
+    if (outgoing.headersSent) {
+      outgoing.destroy()
+      console.info('The response has already been sent.')
+      return
+    }
+
     // now we can write the response headers and status
     outgoing.writeHead(res.status, resHeaderRecord)