@nxtedition/lib 23.3.18 → 23.3.19

package/http.js

@@ -251,135 +251,6 @@ export async function requestMiddleware(ctx, next) {
   }
 }
 
-export async function request(ctx, next) {
-  const { req, res, logger } = ctx
-  const startTime = performance.now()
-
-  const ac = ctx.signal !== undefined ? null : new AbortController()
-
-  let reqLogger = logger
-
-  pendingSet.add(ctx)
-  try {
-    ctx.url = requestTarget(req)
-    if (!ctx.url) {
-      throw new createError.BadRequest('invalid url')
-    }
-
-    ctx.id = req.id = res.id = req.headers['request-id'] || genReqId()
-    ctx.userAgent = req['user-agent'] || req['User-Agent'] || ''
-    ctx.method = req.method
-    ctx.query = ctx.url.search.length > 1 ? querystring.parse(ctx.url.search.slice(1)) : {}
-    ctx.logger =
-      req.log =
-      res.log =
-        logger.child({ req: { id: req.id, url: req.url, userAgent: ctx.userAgent } })
-
-    if (ac) {
-      ctx.signal = ac.signal
-    }
-
-    if (req.method === 'GET' || req.method === 'HEAD') {
-      req.resume() // Dump the body if there is one.
-    }
-
-    res.setHeader('request-id', req.id)
-
-    const isHealthcheck = ctx.url.pathname === '/healthcheck' || ctx.url.pathname === '/_up'
-
-    reqLogger = logger.child({ req })
-    if (!isHealthcheck) {
-      reqLogger.debug('request started')
-    }
-
-    const nextPromise = next()
-    const responsePromise = new Promise((resolve, reject) => {
-      res.on('timeout', onTimeout).on('error', reject).on('close', resolve)
-    })
-
-    if (nextPromise) {
-      await Promise.all([nextPromise, responsePromise])
-    } else if (!res.closed) {
-      await responsePromise
-    }
-
-    const elapsedTime = performance.now() - startTime
-
-    if (isHealthcheck) {
-      // Do nothing...
-    } else if (!res.writableEnded) {
-      reqLogger.debug({ res, elapsedTime }, 'request aborted')
-    } else if (res.statusCode >= 500) {
-      reqLogger.error({ res, elapsedTime }, 'request error')
-    } else if (res.statusCode >= 400) {
-      reqLogger.warn({ res, elapsedTime }, 'request failed')
-    } else {
-      reqLogger.debug({ res, elapsedTime }, 'request completed')
-    }
-  } catch (err) {
-    ac?.abort(err)
-
-    const statusCode = err.statusCode || err.$metadata?.httpStatusCode || 500
-    const elapsedTime = performance.now() - startTime
-
-    if (!res.headersSent && !res.destroyed) {
-      res.statusCode = statusCode
-
-      let reqId = req?.id || err.id
-      for (const name of res.getHeaderNames()) {
-        if (!reqId && name === 'request-id') {
-          reqId = res.getHeader(name)
-        }
-        res.removeHeader(name)
-      }
-
-      if (reqId) {
-        res.setHeader('request-id', reqId)
-      }
-
-      if (fp.isPlainObject(err.headers)) {
-        for (const [key, val] of Object.entries(err.headers)) {
-          if (!ERR_HEADER_EXPR.test(key)) {
-            res.setHeader(key, val)
-          }
-        }
-      }
-
-      if (fp.isPlainObject(err.body)) {
-        res.setHeader('content-type', 'application/json')
-        res.end(JSON.stringify(err.body))
-      } else if (typeof err.body === 'string') {
-        res.end(err.body)
-      } else if (Buffer.isBuffer(err.body)) {
-        res.end(err.body)
-      } else {
-        res.end()
-      }
-
-      if (statusCode < 500) {
-        reqLogger.warn({ res, err, elapsedTime }, 'request failed')
-      } else {
-        reqLogger.error({ res, err, elapsedTime }, 'request error')
-      }
-    } else {
-      if (req.aborted || !res.writableEnded || err.name === 'AbortError') {
-        reqLogger.debug({ res, err, elapsedTime }, 'request aborted')
-      } else if (statusCode < 500) {
-        reqLogger.warn({ res, err, elapsedTime }, 'request failed')
-      } else {
-        reqLogger.error({ res, err, elapsedTime }, 'request error')
-      }
-    }
-  } finally {
-    pendingSet.delete(ctx)
-
-    if (!res.writableEnded) {
-      res.destroy()
-      logger.debug('request destroyed')
-    }
-  }
-}
-
 export class ServerResponse extends http.ServerResponse {
   #created = 0
   #bytesWritten = 0
@@ -570,6 +441,137 @@ export class Http2ServerResponse extends http2.Http2ServerResponse {
   }
 }
 
+/** @deprecated */
+export async function request(ctx, next) {
+  const { req, res, logger } = ctx
+  const startTime = performance.now()
+
+  const ac = ctx.signal !== undefined ? null : new AbortController()
+
+  let reqLogger = logger
+
+  pendingSet.add(ctx)
+  try {
+    ctx.url = requestTarget(req)
+    if (!ctx.url) {
+      throw new createError.BadRequest('invalid url')
+    }
+
+    ctx.id = req.id = res.id = req.headers['request-id'] || genReqId()
+    ctx.userAgent = req['user-agent'] || req['User-Agent'] || ''
+    ctx.method = req.method
+    ctx.query = ctx.url.search.length > 1 ? querystring.parse(ctx.url.search.slice(1)) : {}
+    ctx.logger =
+      req.log =
+      res.log =
+        logger.child({ req: { id: req.id, url: req.url, userAgent: ctx.userAgent } })
+
+    if (ac) {
+      ctx.signal = ac.signal
+    }
+
+    if (req.method === 'GET' || req.method === 'HEAD') {
+      req.resume() // Dump the body if there is one.
+    }
+
+    res.setHeader('request-id', req.id)
+
+    const isHealthcheck = ctx.url.pathname === '/healthcheck' || ctx.url.pathname === '/_up'
+
+    reqLogger = logger.child({ req })
+    if (!isHealthcheck) {
+      reqLogger.debug('request started')
+    }
+
+    const nextPromise = next()
+    const responsePromise = new Promise((resolve, reject) => {
+      res.on('timeout', onTimeout).on('error', reject).on('close', resolve)
+    })
+
+    if (nextPromise) {
+      await Promise.all([nextPromise, responsePromise])
+    } else if (!res.closed) {
+      await responsePromise
+    }
+
+    const elapsedTime = performance.now() - startTime
+
+    if (isHealthcheck) {
+      // Do nothing...
+    } else if (!res.writableEnded) {
+      reqLogger.debug({ res, elapsedTime }, 'request aborted')
+    } else if (res.statusCode >= 500) {
+      reqLogger.error({ res, elapsedTime }, 'request error')
+    } else if (res.statusCode >= 400) {
+      reqLogger.warn({ res, elapsedTime }, 'request failed')
+    } else {
+      reqLogger.debug({ res, elapsedTime }, 'request completed')
+    }
+  } catch (err) {
+    ac?.abort(err)
+
+    const statusCode = err.statusCode || err.$metadata?.httpStatusCode || 500
+    const elapsedTime = performance.now() - startTime
+
+    if (!res.headersSent && !res.destroyed) {
+      res.statusCode = statusCode
+
+      let reqId = req?.id || err.id
+      for (const name of res.getHeaderNames()) {
+        if (!reqId && name === 'request-id') {
+          reqId = res.getHeader(name)
+        }
+        res.removeHeader(name)
+      }
+
+      if (reqId) {
+        res.setHeader('request-id', reqId)
+      }
+
+      if (fp.isPlainObject(err.headers)) {
+        for (const [key, val] of Object.entries(err.headers)) {
+          if (!ERR_HEADER_EXPR.test(key)) {
+            res.setHeader(key, val)
+          }
+        }
+      }
+
+      if (fp.isPlainObject(err.body)) {
+        res.setHeader('content-type', 'application/json')
+        res.end(JSON.stringify(err.body))
+      } else if (typeof err.body === 'string') {
+        res.end(err.body)
+      } else if (Buffer.isBuffer(err.body)) {
+        res.end(err.body)
+      } else {
+        res.end()
+      }
+
+      if (statusCode < 500) {
+        reqLogger.warn({ res, err, elapsedTime }, 'request failed')
+      } else {
+        reqLogger.error({ res, err, elapsedTime }, 'request error')
+      }
+    } else {
+      if (req.aborted || !res.writableEnded || err.name === 'AbortError') {
+        reqLogger.debug({ res, err, elapsedTime }, 'request aborted')
+      } else if (statusCode < 500) {
+        reqLogger.warn({ res, err, elapsedTime }, 'request failed')
+      } else {
+        reqLogger.error({ res, err, elapsedTime }, 'request error')
+      }
+    }
+  } finally {
+    pendingSet.delete(ctx)
+
+    if (!res.writableEnded) {
+      res.destroy()
+      logger.debug('request destroyed')
+    }
+  }
+}
+
+/** @deprecated */
 export function createServer(options, ctx, middleware) {
   middleware = Array.isArray(middleware) ? middleware : [middleware]
   middleware = fp.values(middleware)
@@ -598,6 +600,7 @@ export function createServer(options, ctx, middleware) {
   return server
 }
 
+/** @deprecated */
 export async function upgrade(ctx, next) {
   const { req, res, socket = res, logger } = ctx
 
@@ -674,6 +677,7 @@ export async function upgrade(ctx, next) {
   }
 }
 
+/** @deprecated */
 export function isConnectionError(err) {
   // AWS compat.
   const statusCode = err?.statusCode ?? err?.$metadata?.httpStatusCode
@@ -696,6 +700,7 @@ export function isConnectionError(err) {
     : false
 }
 
+/** @deprecated */
 export function defaultDelay(err, retryCount, options) {
   const { signal, logger = null } = options ?? {}
   if (isConnectionError(err)) {
@@ -708,6 +713,7 @@ export function defaultDelay(err, retryCount, options) {
   }
 }
 
+/** @deprecated */
 export async function retry(fn, options) {
   const { maxRetries = 8, count = maxRetries, delay = defaultDelay, signal } = options ?? {}
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nxtedition/lib",
-  "version": "23.3.18",
+  "version": "23.3.19",
   "license": "MIT",
   "author": "Robert Nagy <robert.nagy@boffins.se>",
   "type": "module",

package/serializers.js

@@ -217,11 +217,21 @@ Object.defineProperty(pinoErrProto, rawSymbol, {
 function errSerializer(err) {
   if (Array.isArray(err)) {
     return err.length === 0 ? undefined : errSerializer({ message: '', errors: err })
-  } else if (!isErrorLike(err)) {
+  }
+
+  if (!isErrorLike(err)) {
     return undefined
-  } else if (err.message === '' && Array.isArray(err.errors) && err.errors.length === 0) {
+  }
+
+  const errors = Array.isArray(err?.errors)
+    ? err.errors.map((err) => errSerializer(err)).filter(Boolean)
+    : null
+
+  if (err.message === '' && errors != null && errors.length === 0) {
     return undefined
-  } else if (err.message === '' && Array.isArray(err.errors) && err.errors.length === 1) {
+  }
+
+  if (err.message === '' && errors != null && errors.length === 1) {
     return errSerializer(err.errors[0])
   }
 
@@ -233,8 +243,8 @@ function errSerializer(err) {
   _err.message = err.message
   _err.stack = err.stack
 
-  if (Array.isArray(err.errors)) {
-    _err.aggregateErrors = err.errors.map((err) => errSerializer(err))
+  if (errors != null) {
+    _err.aggregateErrors = errors
   }
 
   if (isErrorLike(err.cause) && !Object.prototype.hasOwnProperty.call(err.cause, seen)) {