@nxtedition/nxt-undici 6.2.14 → 6.2.16

package/lib/interceptor/response-retry.js

@@ -4,7 +4,6 @@ import { DecoratorHandler, isDisturbed, decorateError, parseRangeHeader } from '
 
 function noop() {}
 
-// TODO (fix): What about onUpgrade?
 class Handler extends DecoratorHandler {
   #dispatch
   #opts
@@ -18,6 +17,7 @@ class Handler extends DecoratorHandler {
   #headers
   #trailers
   #body
+  #bodySize = 0
 
   #abort
   #aborted = false
@@ -48,6 +48,7 @@ class Handler extends DecoratorHandler {
     this.#statusCode = 0
     this.#headers = null
     this.#body = null
+    this.#bodySize = 0
     this.#trailers = null
 
     if (!this.#headersSent) {
@@ -75,6 +76,11 @@ class Handler extends DecoratorHandler {
     }
   }
 
+  onUpgrade() {
+    // TODO (fix): Should we support this?
+    throw new Error('not supported')
+  }
+
   onHeaders(statusCode, headers, resume) {
     this.#statusCode = statusCode
     this.#headers = headers
@@ -121,6 +127,7 @@ class Handler extends DecoratorHandler {
         this.#etag = headers.etag
       } else if (statusCode >= 400) {
         this.#body = []
+        this.#bodySize = 0
         return true
       } else {
         this.#headersSent = true
@@ -146,13 +153,13 @@ class Handler extends DecoratorHandler {
 
       if (this.#pos > 0 && this.#etag !== headers.etag) {
         this.#maybeError(null)
-        return null
+        return false
       }
 
       const contentRange = parseRangeHeader(headers['content-range'])
       if (!contentRange) {
         this.#maybeError(null)
-        return null
+        return false
       }
 
       const { start, size, end = size } = contentRange
@@ -165,6 +172,7 @@ class Handler extends DecoratorHandler {
       return true
     } else {
       this.#maybeError(this.#retryError)
+      return false
     }
   }
 
@@ -177,7 +185,14 @@ class Handler extends DecoratorHandler {
       return super.onData(chunk)
     }
 
-    this.#body?.push(chunk)
+    if (this.#body) {
+      this.#body.push(chunk)
+      this.#bodySize += chunk.byteLength
+      if (this.#bodySize > 256 * 1024) {
+        this.#body = null
+        this.#bodySize = 0
+      }
+    }
   }
 
   onComplete(trailers) {
@@ -351,6 +366,7 @@ class Handler extends DecoratorHandler {
 export default () => (dispatch) => (opts, handler) =>
   opts.retry !== false &&
   !opts.upgrade &&
-  (/^(HEAD|GET|PUT|PATCH)$/.test(opts.method) || opts.idempotent)
+  (/^(HEAD|GET|PUT|PATCH|QUERY)$/.test(opts.method) || opts.idempotent) &&
+  opts.idempotent !== false
     ? dispatch(opts, new Handler(opts, { handler, dispatch }))
     : dispatch(opts, handler)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nxtedition/nxt-undici",
-  "version": "6.2.14",
+  "version": "6.2.16",
   "license": "MIT",
   "author": "Robert Nagy <robert.nagy@boffins.se>",
   "main": "lib/index.js",