fix: prevent duplicate logs

src/http/plugins/log-request.ts

@@ -18,7 +18,7 @@
 
   interface FastifyContextConfig {
     operation?: { type: string }
     resources?: (req: FastifyRequest<any>) => string[]
   }
 }
 
@@ -32,16 +32,20 @@
       fastify.addHook('onRequest', async (req, res) => {
         req.startTime = Date.now()
 
-        // Request was aborted before the server finishes to return a response
         res.raw.once('close', () => {
-          const aborted = !res.raw.writableFinished
-          if (aborted) {
+          if (req.raw.aborted) {
             doRequestLog(req, {
               excludeUrls: options.excludeUrls,
-              statusCode: 'ABORTED RES',
+              statusCode: 'ABORTED REQ',
               responseTime: (Date.now() - req.startTime) / 1000,
             })
+            return
           }
+          doRequestLog(req, {
+            excludeUrls: options.excludeUrls,
+            statusCode: 'ABORTED RES',
+            responseTime: (Date.now() - req.startTime) / 1000,
+          })
         })
       })
 
@@ -53,7 +57,7 @@
         const resources = getFirstDefined<string[]>(
           req.resources,
           req.routeConfig.resources?.(req),
           (req.raw as any).resources,
           resourceFromParams ? [resourceFromParams] : ([] as string[])
         )
 
@@ -73,14 +77,6 @@
         }
       })
 
-      fastify.addHook('onRequestAbort', async (req) => {
-        doRequestLog(req, {
-          excludeUrls: options.excludeUrls,
-          statusCode: 'ABORTED REQ',
-          responseTime: (Date.now() - req.startTime) / 1000,
-        })
-      })
-
       fastify.addHook('onResponse', async (req, reply) => {
         doRequestLog(req, {
           reply,
@@ -116,7 +112,7 @@
   const rId = req.id
   const cIP = req.ip
   const statusCode = options.statusCode
   const error = (req.raw as any).executionError || req.executionError
   const tenantId = req.tenantId
 
   const buildLogMessage = `${tenantId} | ${rMeth} | ${statusCode} | ${cIP} | ${rId} | ${rUrl} | ${uAgent}`

src/http/plugins/signals.ts

@@ -20,6 +20,17 @@ export const signals = fastifyPlugin(
         disconnect: new AbortController(),
       }
 
+      // Client terminated the request before the body was fully sent
+      req.raw.once('close', () => {
+        if (req.raw.aborted) {
+          req.signals.body.abort()
+
+          if (!req.signals.disconnect.signal.aborted) {
+            req.signals.disconnect.abort()
+          }
+        }
+      })
+
       // Client terminated the request before server finished sending the response
       res.raw.once('close', () => {
         const aborted = !res.raw.writableFinished
@@ -33,7 +44,6 @@ export const signals = fastifyPlugin(
       })
     })
 
-    // Client terminated the request before the body was fully sent
     fastify.addHook('onRequestAbort', async (req) => {
       req.signals.body.abort()