@nxtedition/lib 21.2.2 → 21.3.1

package/couch.js

@@ -117,6 +117,7 @@ export function makeCouch(opts) {
    * @param {number} [options.heartbeat=60000] - The interval at which to send a heartbeat.
    * @param {function} [options.retry=null] - The function to retry the request on error.
    * @param {string} [options.since=null] - The sequence number to start from.
+   * @param {string} [options.highWaterMark=128 * 1024] - Buffering.
    * @yields {Array<{ id: string, seq?: string, doc?: Object, deleted?: boolean, changes: Array<{ rev: string }> }>}
    */
   async function* changes({ client = defaultClient, signal = null, logger, ...options } = {}) {
@@ -207,6 +208,7 @@ export function makeCouch(opts) {
       }
     }
 
+    const highWaterMark = options.highWaterMark
     const live = options.live == null || !!options.live
     const retry =
       options.retry ??
@@ -220,6 +222,7 @@ export function makeCouch(opts) {
         method,
         live,
         retry,
+        highWaterMark,
         params,
         body,
         signal: ac.signal,
@@ -245,6 +248,7 @@ export function makeCouch(opts) {
     body,
     signal,
     client,
+    highWaterMark = 128 * 1024,
     blocking = live || !params.limit || params.limit > 256,
   }) {
     let retryCount = 0
@@ -270,7 +274,7 @@ export function makeCouch(opts) {
           signal,
           headersTimeout: 2 * 60e3,
           bodyTimeout: 2 * (params.heartbeat || 60e3),
-          highWaterMark: 128 * 1024,
+          highWaterMark,
           dispatcher: client,
         }
 

package/http.js

@@ -116,7 +116,9 @@ export async function request2(ctx, next) {
 
     const isHealthcheck = req.url === '/healthcheck' || req.url === '/_up'
 
-    if (!isHealthcheck) {
+    if (isHealthcheck) {
+      // Do nothing...
+    } else {
       logger.debug({ req }, 'request started')
     }
 
@@ -132,6 +134,7 @@ export async function request2(ctx, next) {
       // Do nothing...
     } else if (!res.writableEnded) {
       logger.debug({ res, elapsedTime }, 'request aborted')
+      res.destroy()
     } else if (res.statusCode >= 500) {
       logger.error({ res, elapsedTime }, 'request error')
     } else if (res.statusCode >= 400) {
@@ -143,12 +146,12 @@ export async function request2(ctx, next) {
     ctx[kAbortController]?.abort(err)
 
     const statusCode = err.statusCode || err.$metadata?.httpStatusCode || 500
-    const responseTime = Math.ceil(performance.now() - startTime)
+    const elapsedTime = Math.ceil(performance.now() - startTime)
 
     if (!res.headersSent && !res.destroyed) {
       res.statusCode = statusCode
 
-      let reqId = req?.id || err.id
+      let reqId = ctx.id || req?.id || err.id
       for (const name of res.getHeaderNames()) {
         if (!reqId && name === 'request-id') {
           reqId = res.getHeader(name)
@@ -160,8 +163,20 @@ export async function request2(ctx, next) {
         res.setHeader('request-id', reqId)
       }
 
-      if (fp.isPlainObject(err.headers)) {
-        for (const [key, val] of Object.entries(err.headers)) {
+      const { headers } = err
+
+      if (fp.isPlainObject(headers)) {
+        for (const [key, val] of Object.entries(headers)) {
+          if (!ERR_HEADER_EXPR.test(key)) {
+            res.setHeader(key, val)
+          }
+        }
+      } else if (Array.isArray(err.headers)) {
+        assert(headers.length % 2 === 0)
+        assert(headers.length === 0 || typeof headers[0] === 'string')
+        for (let n = 0; n < headers.length; n += 2) {
+          const key = headers[n + 0]
+          const val = headers[n + 1]
           if (!ERR_HEADER_EXPR.test(key)) {
             res.setHeader(key, val)
           }
@@ -170,23 +185,27 @@ export async function request2(ctx, next) {
 
       if (fp.isPlainObject(err.body)) {
         res.setHeader('content-type', 'application/json')
-        res.write(JSON.stringify(err.body))
+        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) {
-        logger.warn({ req, res, err, responseTime }, 'request failed')
+        logger.warn({ req, res, err, elapsedTime }, 'request failed')
       } else {
-        logger.error({ req, res, err, responseTime }, 'request error')
+        logger.error({ req, res, err, elapsedTime }, 'request error')
       }
-
-      res.end()
     } else {
       if (req.aborted || !res.writableEnded || err.name === 'AbortError') {
-        logger.debug({ req, res, err, responseTime }, 'request aborted')
+        logger.debug({ req, res, err, elapsedTime }, 'request aborted')
       } else if (statusCode < 500) {
-        logger.warn({ req, res, err, responseTime }, 'request failed')
+        logger.warn({ req, res, err, elapsedTime }, 'request failed')
       } else {
-        logger.error({ req, res, err, responseTime }, 'request error')
+        logger.error({ req, res, err, elapsedTime }, 'request error')
       }
 
       if (!res.writableEnded) {
@@ -248,6 +267,7 @@ export async function request(ctx, next) {
       // Do nothing...
     } else if (!res.writableEnded) {
       reqLogger.debug({ res, elapsedTime }, 'request aborted')
+      res.destroy()
     } else if (res.statusCode >= 500) {
       reqLogger.error({ res, elapsedTime }, 'request error')
     } else if (res.statusCode >= 400) {
@@ -259,7 +279,7 @@ export async function request(ctx, next) {
     ac?.abort(err)
 
     const statusCode = err.statusCode || err.$metadata?.httpStatusCode || 500
-    const responseTime = Math.ceil(performance.now() - startTime)
+    const elapsedTime = Math.ceil(performance.now() - startTime)
 
     if (!res.headersSent && !res.destroyed) {
       res.statusCode = statusCode
@@ -286,23 +306,27 @@ export async function request(ctx, next) {
 
       if (fp.isPlainObject(err.body)) {
         res.setHeader('content-type', 'application/json')
-        res.write(JSON.stringify(err.body))
+        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, responseTime }, 'request failed')
+        reqLogger.warn({ res, err, elapsedTime }, 'request failed')
       } else {
-        reqLogger.error({ res, err, responseTime }, 'request error')
+        reqLogger.error({ res, err, elapsedTime }, 'request error')
       }
-
-      res.end()
     } else {
       if (req.aborted || !res.writableEnded || err.name === 'AbortError') {
-        reqLogger.debug({ res, err, responseTime }, 'request aborted')
+        reqLogger.debug({ res, err, elapsedTime }, 'request aborted')
       } else if (statusCode < 500) {
-        reqLogger.warn({ res, err, responseTime }, 'request failed')
+        reqLogger.warn({ res, err, elapsedTime }, 'request failed')
       } else {
-        reqLogger.error({ res, err, responseTime }, 'request error')
+        reqLogger.error({ res, err, elapsedTime }, 'request error')
       }
 
       if (!res.writableEnded) {

package/package.json

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