@nxtedition/nxt-undici 5.1.2 → 5.1.3

package/lib/index.js

@@ -72,13 +72,11 @@ function wrapDispatch(dispatcher) {
           headers['user-agent'] = userAgent
         }
 
-        const url = opts.url ? new URL(opts.url) : null
-
         return dispatch(
           {
             id: opts.id,
-            origin: opts.origin ?? url?.origin,
-            path: opts.path ?? (url?.search ? `${url.pathname}${url.search}` : url?.pathname),
+            origin: opts.origin,
+            path: opts.path,
             method: opts.method ?? (opts.body ? 'POST' : 'GET'),
             body: opts.body,
             query: opts.query,

package/lib/interceptor/log.js

@@ -17,12 +17,15 @@ class Handler extends DecoratorHandler {
     end: -1,
   }
 
+  #statusCode
+  #headers
+
   constructor(opts, { handler }) {
     super(handler)
 
     this.#handler = handler
     this.#opts = opts
-    this.#logger = opts.logger
+    this.#logger = opts.logger.#logger.child({ ureq: opts })
 
     this.#created = performance.now()
   }
@@ -36,7 +39,6 @@ class Handler extends DecoratorHandler {
     this.#timing.data = -1
     this.#timing.end = -1
 
-    this.#logger = this.#logger.child({ ureq: this.#opts })
     this.#logger.debug('upstream request started')
 
     return this.#handler.onConnect((reason) => {
@@ -50,7 +52,6 @@ class Handler extends DecoratorHandler {
 
     this.#logger.debug(
       {
-        ureq: { id: this.#opts.id, url: String(this.#opts.url) },
         ures: { statusCode, headers },
         elapsedTime: this.#timing.headers,
       },
@@ -67,14 +68,8 @@ class Handler extends DecoratorHandler {
   onHeaders(statusCode, rawHeaders, resume, statusMessage, headers = parseHeaders(rawHeaders)) {
     this.#timing.headers = performance.now() - this.#created
 
-    this.#logger.debug(
-      {
-        ureq: { id: this.#opts.id, url: String(this.#opts.url) },
-        ures: { statusCode, headers },
-        elapsedTime: this.#timing.headers,
-      },
-      'upstream request response',
-    )
+    this.#statusCode = statusCode
+    this.#headers = headers
 
     return this.#handler.onHeaders(statusCode, null, resume, statusMessage, headers)
   }
@@ -95,9 +90,11 @@ class Handler extends DecoratorHandler {
     this.#logger.debug(
       {
         ures: {
+          statusCode: this.#statusCode,
+          headers: this.#headers,
+          timing: this.#timing,
           bytesRead: this.#pos,
           bytesReadPerSecond: (this.#pos * 1e3) / (this.#timing.end - this.#timing.data),
-          timing: this.#timing,
         },
         elapsedTime: this.#timing.end,
       },
@@ -111,10 +108,13 @@ class Handler extends DecoratorHandler {
     this.#timing.end = performance.now() - this.#created
 
     const data = {
+      ureq: this.#opts,
       ures: {
+        statusCode: this.#statusCode,
+        headers: this.#headers,
+        timing: this.#timing,
         bytesRead: this.#pos,
         bytesReadPerSecond: (this.#pos * 1e3) / (this.#timing.end - this.#timing.data),
-        timing: this.#timing,
       },
       elapsedTime: this.#timing.end,
       err,

package/lib/request.js

@@ -158,7 +158,6 @@ export function request(dispatch, url, opts) {
   }
 
   let origin = url.origin
-
   if (!origin) {
     const protocol = url.protocol ?? 'http:'
     const host = url.host ?? `${url.hostname}:${url.port ?? (protocol === 'https:' ? 443 : 80)}`
@@ -167,7 +166,7 @@ export function request(dispatch, url, opts) {
 
   let path = url.path
   if (!path) {
-    path = url.search ? `${url.pathname}${url.search}` : url
+    path = url.search ? `${url.pathname}${url.search}` : url.pathname
   }
 
   opts = { ...opts, origin, path }

package/package.json

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