@telorun/http-server 0.8.0 → 0.8.1

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @telorun/http-server
 
+## 0.8.1
+
+### Patch Changes
+
+- e3146f3: Fix spurious request cancellation (HTTP 499) for body-bearing requests whose handler awaits before replying. Per-request cancellation was wired to the request stream's `close` event, which fires as normal cleanup once a request body has been fully received — so any `PUT`/`POST` whose handler did async work (e.g. a DB query) before sending a response was cancelled mid-flight and answered with 499. Cancellation now listens on the response socket, which only closes early on a genuine client disconnect; normal completions and synchronous rejects are unaffected.
+
 ## 0.8.0
 
 ### Minor Changes

package/dist/http-api-controller.js

@@ -107,9 +107,13 @@ export class HttpServerApi {
                 };
                 const sink = fastifyReplySink(reply);
                 // Per-request cancellation: abandon downstream work when the client
-                // disconnects before the response is sent.
+                // disconnects before the response is sent. Listen on the response
+                // socket, not the request stream — the latter's `close` fires as normal
+                // cleanup once a request body has been fully received, which would
+                // cancel any body-bearing request that awaits (e.g. a DB call) before
+                // replying. The response socket only closes early on a real disconnect.
                 const cancellation = this.ctx.createCancellationSource();
-                request.raw.on("close", () => {
+                reply.raw.on("close", () => {
                     if (!reply.sent)
                         cancellation.cancel("client-disconnect");
                 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@telorun/http-server",
-  "version": "0.8.0",
+  "version": "0.8.1",
   "description": "Telo HTTP Server module - HTTP server and API resource kinds for Telo manifests.",
   "keywords": [
     "telo",

package/src/http-api-controller.ts

@@ -148,9 +148,13 @@ export class HttpServerApi implements ResourceInstance {
         const sink = fastifyReplySink(reply);
 
         // Per-request cancellation: abandon downstream work when the client
-        // disconnects before the response is sent.
+        // disconnects before the response is sent. Listen on the response
+        // socket, not the request stream — the latter's `close` fires as normal
+        // cleanup once a request body has been fully received, which would
+        // cancel any body-bearing request that awaits (e.g. a DB call) before
+        // replying. The response socket only closes early on a real disconnect.
         const cancellation = this.ctx.createCancellationSource();
-        request.raw.on("close", () => {
+        reply.raw.on("close", () => {
           if (!reply.sent) cancellation.cancel("client-disconnect");
         });