@nxtedition/lib 15.0.42 → 15.0.43

package/http.js

@@ -7,7 +7,6 @@ const compose = require('koa-compose')
 const http = require('http')
 const fp = require('lodash/fp')
 const tp = require('timers/promises')
-const { AbortError } = require('./errors')
 
 const ERR_HEADER_EXPR =
   /^(content-length|content-type|te|host|upgrade|trailers|connection|keep-alive|http2-settings|transfer-encoding|proxy-connection|proxy-authenticate|proxy-authorization)$/i
@@ -60,19 +59,34 @@ module.exports.request = async function request(ctx, next) {
       reqLogger.trace({ req }, 'request started')
     }
 
-    const onClose = () => queueMicrotask(() => ac.abort())
-    const onTimeout = () => res.destroy(new createError.RequestTimeout())
-    const onError = (err) => ac.abort(err)
-
-    res.on('close', onClose).on('timeout', onTimeout).on('error', onError)
-
-    req.on('timeout', onTimeout).on('error', onError)
-
-    await next()
+    await Promise.all([
+      next(),
+      new Promise((resolve, reject) => {
+        res
+          .on('timeout', function () {
+            this.destroy(new createError.RequestTimeout())
+          })
+          .on('error', (err) => {
+            reqLogger.error({ err }, 'response error')
+            reject(err)
+          })
+          .on('close', () => {
+            reqLogger.debug('response closed')
+            resolve(null)
+          })
 
-    if (!res.writableEnded && res.destroyed) {
-      throw new AbortError()
-    }
+        req
+          .on('timeout', function () {
+            this.destroy(new createError.RequestTimeout())
+          })
+          .on('error', (err) => {
+            reqLogger.error({ err }, 'request error')
+          })
+          .on('close', () => {
+            reqLogger.debug('request closed')
+          })
+      }),
+    ])
 
     assert(res.writableEnded)
     assert(res.statusCode)
@@ -90,6 +104,8 @@ module.exports.request = async function request(ctx, next) {
     } else {
       reqLogger.trace('request completed')
     }
+
+    ac.abort()
   } catch (err) {
     const reason = ac.signal.reason
     const responseTime = Math.round(performance.now() - startTime)
@@ -161,6 +177,8 @@ module.exports.request = async function request(ctx, next) {
         res.destroy()
       }
     }
+
+    ac.abort(err)
   }
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nxtedition/lib",
-  "version": "15.0.42",
+  "version": "15.0.43",
   "license": "MIT",
   "author": "Robert Nagy <robert.nagy@boffins.se>",
   "files": [