@nxtedition/nxt-undici 3.3.2 → 3.3.3

package/lib/interceptor/cache.js

@@ -193,7 +193,9 @@ export default (opts) => (dispatch) => (opts, handler) => {
     } catch (err) {
       handler.onError(err)
     }
+
+    return true
   } else {
-    dispatch(opts, new CacheHandler({ handler, store, key: makeKey(opts) }))
+    return dispatch(opts, new CacheHandler({ handler, store, key: makeKey(opts) }))
   }
 }

package/lib/interceptor/dns.js

@@ -60,35 +60,37 @@ export default (interceptorOpts) => (dispatch) => (opts, handler) => {
   const { hostname } = new URL(opts.origin)
 
   if (net.isIP(hostname)) {
-    dispatch(opts, handler)
-  } else {
-    const callback = (err, val) => {
-      if (err) {
-        handler.onConnect(() => {})
-        handler.onError(err)
-      } else {
-        const url = new URL(opts.origin)
-        url.hostname = Array.isArray(val)
-          ? val[Math.floor(val.length * Math.random())].address
-          : val?.address ?? val
-        dispatch(
-          { ...opts, origin: url.origin },
-          resolver.clear ? new Handler({ resolver, key: hostname }, { handler }) : handler,
-        )
-      }
-    }
+    return dispatch(opts, handler)
+  }
 
-    try {
-      const thenable = resolver.lookup(hostname, { family, hints, order, all }, callback)
-      if (typeof thenable?.then === 'function') {
-        thenable.then(
-          (val) => callback(null, val),
-          (err) => callback(err),
-        )
-      }
-    } catch (err) {
+  const callback = (err, val) => {
+    if (err) {
       handler.onConnect(() => {})
       handler.onError(err)
+    } else {
+      const url = new URL(opts.origin)
+      url.hostname = Array.isArray(val)
+        ? val[Math.floor(val.length * Math.random())].address
+        : val?.address ?? val
+      dispatch(
+        { ...opts, origin: url.origin },
+        resolver.clear ? new Handler({ resolver, key: hostname }, { handler }) : handler,
+      )
     }
   }
+
+  try {
+    const thenable = resolver.lookup(hostname, { family, hints, order, all }, callback)
+    if (typeof thenable?.then === 'function') {
+      thenable.then(
+        (val) => callback(null, val),
+        (err) => callback(err),
+      )
+    }
+  } catch (err) {
+    handler.onConnect(() => {})
+    handler.onError(err)
+  }
+
+  return true
 }

package/lib/interceptor/lookup.js

@@ -25,4 +25,6 @@ export default (opts) => (dispatch) => (opts, handler) => {
   } catch (err) {
     callback(err)
   }
+
+  return true
 }

package/lib/interceptor/request-body-factory.js

@@ -16,11 +16,13 @@ export default (opts) => (dispatch) => (opts, handler) => {
           handler.onError(err)
         },
       )
+      return true
     } else {
-      dispatch({ ...opts, body }, handler)
+      return dispatch({ ...opts, body }, handler)
     }
   } catch (err) {
     handler.onConnect(() => {})
     handler.onError(err)
+    return true
   }
 }

package/lib/utils.js

@@ -3,6 +3,8 @@ import cacheControlParser from 'cache-control-parser'
 import stream from 'node:stream'
 import { util } from '@nxtedition/undici'
 
+const noop = () => {}
+
 export function parseCacheControl(str) {
   return str ? cacheControlParser.parse(str) : null
 }
@@ -267,7 +269,7 @@ export class DecoratorHandler {
   onError(...args) {
     if (!this.#onConnectCalled) {
       this.#onConnectCalled = true
-      this.#handler.onConnect?.(...args)
+      this.#handler.onConnect?.(noop)
     }
     return this.#handler.onError?.(...args)
   }

package/package.json

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