undici 4.11.3 → 4.12.0

package/README.md

@@ -203,13 +203,13 @@ not support or does not fully implement.
 * https://fetch.spec.whatwg.org/#garbage-collection
 
 The [Fetch Standard](https://fetch.spec.whatwg.org) allows users to skip consuming the response body by relying on
-[garbage collection](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Memory_Management#garbage_collection) to release connection resources. Undici does the same. However,
-garbage collection in Node is less aggressive and deterministic (due to the lack
-of clear idle periods that browser have through the rendering refresh rate)
-which means that leaving the release of connection resources to the garbage collector
-can lead to excessive connection usage, reduced performance (due to less connection re-use),
-and even stalls or deadlocks when running out of connections. Therefore, it is highly
-recommended to always either consume or cancel the response body.
+[garbage collection](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Memory_Management#garbage_collection) to release connection resources. Undici does not do the same. Therefore, it is important to always either consume or cancel the response body.
+
+Garbage collection in Node is less aggressive and deterministic
+(due to the lack of clear idle periods that browser have through the rendering refresh rate)
+which means that leaving the release of connection resources to the garbage collector can lead
+to excessive connection usage, reduced performance (due to less connection re-use), and even
+stalls or deadlocks when running out of connections.
 
 ```js
 // Do
@@ -289,7 +289,7 @@ pipeline requests, without checking whether the connection is persistent.
 Hence, automatic fallback to HTTP/1.0 or HTTP/1.1 without pipelining is
 not supported.
 
-Undici will immediately pipeline when retrying requests afters a failed
+Undici will immediately pipeline when retrying requests after a failed
 connection. However, Undici will not retry the first remaining requests in
 the prior pipeline and instead error the corresponding callback/promise/stream.
 

package/lib/core/util.js

@@ -150,7 +150,7 @@ function isDestroyed (stream) {
   return !stream || !!(stream.destroyed || stream[kDestroyed])
 }
 
-function isAborted (stream) {
+function isReadableAborted (stream) {
   const state = stream && stream._readableState
   return isDestroyed(stream) && state && !state.endEmitted
 }
@@ -244,15 +244,28 @@ function validateHandler (handler, method, upgrade) {
 // A body is disturbed if it has been read from and it cannot
 // be re-used without losing state or data.
 function isDisturbed (body) {
-  const state = body && body._readableState
   return !!(body && (
-    (stream.isDisturbed && stream.isDisturbed(body)) ||
-    body[kBodyUsed] ||
-    body.readableDidRead || (state && state.dataEmitted) ||
-    isAborted(body)
+    stream.isDisturbed
+      ? stream.isDisturbed(body) || body[kBodyUsed] // TODO (fix): Why is body[kBodyUsed] needed?
+      : body[kBodyUsed] ||
+        body.readableDidRead ||
+        (body._readableState && body._readableState.dataEmitted) ||
+        isReadableAborted(body)
   ))
 }
 
+function isErrored (body) {
+  return !!(body && (
+    stream.isErrored
+      ? stream.isErrored(body)
+      : /state: 'errored'/.test(nodeUtil.inspect(body)
+  )))
+}
+
+function isReadable (body) {
+  return !!(body && /state: 'readable'/.test(nodeUtil.inspect(body)))
+}
+
 function getSocketInfo (socket) {
   return {
     localAddress: socket.localAddress,
@@ -310,8 +323,10 @@ module.exports = {
   kEnumerableProperty,
   nop,
   isDisturbed,
+  isErrored,
+  isReadable,
   toUSVString: nodeUtil.toUSVString || ((val) => `${val}`),
-  isAborted,
+  isReadableAborted,
   isBlobLike,
   parseOrigin,
   parseURL,

package/lib/fetch/body.js

@@ -3,12 +3,13 @@
 const util = require('../core/util')
 const { ReadableStreamFrom, toUSVString, isBlobLike } = require('./util')
 const { FormData } = require('./formdata')
-const { kState, kError } = require('./symbols')
+const { kState } = require('./symbols')
 const { Blob } = require('buffer')
 const { kBodyUsed } = require('../core/symbols')
 const assert = require('assert')
 const nodeUtil = require('util')
 const { NotSupportedError } = require('../core/errors')
+const { isErrored } = require('../core/util')
 
 let ReadableStream
 
@@ -187,7 +188,7 @@ function extractBody (object, keepalive = false) {
           // Whenever one or more bytes are available and stream is not errored,
           // enqueue a Uint8Array wrapping an ArrayBuffer containing the available
           // bytes into stream.
-          if (!/state: 'errored'/.test(nodeUtil.inspect(stream))) {
+          if (!isErrored(stream)) {
             controller.enqueue(new Uint8Array(value))
           }
         }
@@ -268,10 +269,6 @@ const methods = {
     if (this[kState].body) {
       const stream = this[kState].body.stream
 
-      if (stream[kError]) {
-        throw stream[kError]
-      }
-
       if (util.isDisturbed(stream)) {
         throw new TypeError('disturbed')
       }
@@ -352,19 +349,12 @@ const properties = {
   }
 }
 
-function cancelBody (body, reason) {
-  if (body.stream && !/state: 'errored'/.test(nodeUtil.inspect(body.stream))) {
-    body.stream.cancel(reason)
-  }
-}
-
 function mixinBody (prototype) {
   Object.assign(prototype, methods)
   Object.defineProperties(prototype, properties)
 }
 
 module.exports = {
-  cancelBody,
   extractBody,
   safelyExtractBody,
   cloneBody,

package/lib/fetch/index.js

@@ -24,17 +24,17 @@ const {
   requestCurrentURL,
   setRequestReferrerPolicyOnRedirect,
   tryUpgradeRequestToAPotentiallyTrustworthyURL,
-  makeTimingInfo,
+  createOpaqueTimingInfo,
   appendFetchMetadata,
   corsCheck,
   crossOriginResourcePolicyCheck,
   determineRequestsReferrer,
   coarsenedSharedCurrentTime
 } = require('./util')
-const { kState, kHeaders, kGuard, kRealm, kError } = require('./symbols')
+const { kState, kHeaders, kGuard, kRealm } = require('./symbols')
 const { AbortError } = require('../core/errors')
 const assert = require('assert')
-const { safelyExtractBody, cancelBody } = require('./body')
+const { safelyExtractBody } = require('./body')
 const {
   redirectStatus,
   nullBodyStatus,
@@ -44,14 +44,32 @@ const {
 } = require('./constants')
 const { kHeadersList } = require('../core/symbols')
 const EE = require('events')
-const { PassThrough, pipeline, compose } = require('stream')
+const { PassThrough, pipeline } = require('stream')
+const { isErrored, isReadable } = require('../core/util')
 
 let ReadableStream
 
-// https://fetch.spec.whatwg.org/#garbage-collection
-const registry = new FinalizationRegistry((abort) => {
-  abort()
-})
+class Fetch extends EE {
+  constructor (dispatcher) {
+    super()
+
+    this.dispatcher = dispatcher
+    this.terminated = null
+    this.connection = null
+    this.dump = false
+  }
+
+  terminate ({ reason, aborted } = {}) {
+    if (this.terminated) {
+      return
+    }
+    this.terminated = { aborted, reason }
+
+    this.connection?.destroy(reason)
+
+    this.emit('terminated', reason)
+  }
+}
 
 // https://fetch.spec.whatwg.org/#fetch-method
 async function fetch (...args) {
@@ -73,25 +91,7 @@ async function fetch (...args) {
   const resource = args[0]
   const init = args.length >= 1 ? args[1] ?? {} : {}
 
-  const context = Object.assign(new EE(), {
-    dispatcher: this,
-    terminated: false,
-    connection: null,
-    dump: false,
-    terminate ({ reason, aborted } = {}) {
-      if (this.terminated) {
-        return
-      }
-      this.terminated = { aborted }
-
-      if (this.connection) {
-        this.connection.destroy(reason)
-        this.connection = null
-      }
-
-      this.emit('terminated', reason)
-    }
-  })
+  const context = new Fetch(this)
 
   // 1. Let p be a new promise.
   const p = createDeferredPromise()
@@ -153,7 +153,7 @@ async function fetch (...args) {
   const handleFetchDone = (response) =>
     finalizeAndReportTiming(response, 'fetch')
 
-  // 12. Fetch request with processResponseDone set to handleFetchDone,
+  // 12. Fetch request with processResponseEndOfBody set to handleFetchDone,
   // and processResponse given response being these substeps:
   const processResponse = (response) => {
     // 1. If locallyAborted is true, terminate these substeps.
@@ -193,7 +193,7 @@ async function fetch (...args) {
   fetching
     .call(context, {
       request,
-      processResponseDone: handleFetchDone,
+      processResponseEndOfBody: handleFetchDone,
       processResponse
     })
     .catch((err) => {
@@ -226,11 +226,9 @@ function finalizeAndReportTiming (response, initiatorType = 'other') {
 
   // 6. If response’s timing allow passed flag is not set, then:
   if (!timingInfo.timingAllowPassed) {
-    //  1. Set timingInfo to a new fetch timing info whose start time and
-    //  post-redirect start time are timingInfo’s start time.
-    timingInfo = makeTimingInfo({
-      startTime: timingInfo.startTime,
-      postRedirectStartTime: timingInfo.postRedirectStartTime
+    //  1. Set timingInfo to a the result of creating an opaque timing info for timingInfo.
+    timingInfo = createOpaqueTimingInfo({
+      startTime: timingInfo.startTime
     })
 
     //  2. Set cacheState to the empty string.
@@ -273,8 +271,14 @@ function abortFetch (p, request, responseObject) {
 
   // 3. If request’s body is not null and is readable, then cancel request’s
   // body with error.
-  if (request.body != null) {
-    cancelBody(request.body, error)
+  if (request.body != null && isReadable(request.body?.stream)) {
+    request.body.stream.cancel(error).catch((err) => {
+      if (err.code === 'ERR_INVALID_STATE') {
+        // Node bug?
+        return
+      }
+      throw err
+    })
   }
 
   // 4. If responseObject is null, then return.
@@ -287,13 +291,27 @@ function abortFetch (p, request, responseObject) {
 
   // 6. If response’s body is not null and is readable, then error response’s
   // body with error.
-  if (response.body != null) {
-    cancelBody(response.body, error)
+  if (response.body != null && isReadable(response.body?.stream)) {
+    response.body.stream.cancel(error).catch((err) => {
+      if (err.code === 'ERR_INVALID_STATE') {
+        // Node bug?
+        return
+      }
+      throw err
+    })
   }
 }
 
 // https://fetch.spec.whatwg.org/#fetching
-function fetching ({ request, processResponse, processResponseDone }) {
+function fetching ({
+  request,
+  processRequestBodyChunkLength,
+  processRequestEndOfBody,
+  processResponse,
+  processResponseEndOfBody,
+  processResponseConsumeBody,
+  useParallelQueue = false,
+}) {
   // 1. Let taskDestination be null.
   let taskDestination = null
 
@@ -319,26 +337,28 @@ function fetching ({ request, processResponse, processResponseDone }) {
   // post-redirect start time are the coarsened shared current time given
   // crossOriginIsolatedCapability.
   const currenTime = coarsenedSharedCurrentTime(crossOriginIsolatedCapability)
-  const timingInfo = makeTimingInfo({
-    startTime: currenTime,
-    postRedirectStartTime: currenTime
+  const timingInfo = createOpaqueTimingInfo({
+    startTime: currenTime
   })
 
-  // 6. Let fetchParams be a new fetch params whose request is request, timing
-  // info is timingInfo, process request body is processRequestBody,
-  // process request end-of-body is processRequestEndOfBody, process response
-  // is processResponse, process response end-of-body is
-  // processResponseEndOfBody, process response done is processResponseDone,
-  // task destination is taskDestination, and cross-origin isolated capability
-  // is crossOriginIsolatedCapability.
+  // 6. Let fetchParams be a new fetch params whose
+  // request is request,
+  // timing info is timingInfo,
+  // process request body chunk length is processRequestBodyChunkLength,
+  // process request end-of-body is processRequestEndOfBody,
+  // process response is processResponse,
+  // process response consume body is processResponseConsumeBody,
+  // process response end-of-body is processResponseEndOfBody,
+  // task destination is taskDestination,
+  // and cross-origin isolated capability is crossOriginIsolatedCapability.
   const fetchParams = {
     request,
     timingInfo,
-    processRequestBody: null,
-    processRequestEndOfBody: null,
+    processRequestBodyChunkLength,
+    processRequestEndOfBody,
     processResponse,
-    processResponseEndOfBody: null,
-    processResponseDone,
+    processResponseConsumeBody,
+    processResponseEndOfBody,
     taskDestination,
     crossOriginIsolatedCapability
   }
@@ -656,10 +676,9 @@ async function mainFetch (fetchParams, recursive = false) {
     const processBodyError = (reason) =>
       fetchFinale(fetchParams, makeNetworkError(reason))
 
-    // 2. If request’s response tainting is "opaque", response is a network
-    // error, or response’s body is null, then run processBodyError and abort
-    // these steps.
-    if (request.responseTainting === 'opaque' && response.status === 0) {
+    // 2. If request’s response tainting is "opaque", or response’s body is null,
+    // then run processBodyError and abort these steps.
+    if (request.responseTainting === 'opaque' || response.body == null) {
       processBodyError(response.error)
       return
     }
@@ -717,13 +736,13 @@ function fetchFinale (fetchParams, response) {
     fetchParams.processResponse(response)
   }
 
-  // 2. If fetchParams’s process response end-of-body is non-null, then:.
+  // 2. If fetchParams’s process response consume is non-null, then:.
   //    TODO
   //    1. Let processBody given nullOrBytes be this step: run fetchParams’s
-  //    process response end-of-body given response and nullOrBytes.on.
+  //    process response consume given response and nullOrBytes.on.
   //    TODO
   //    2. Let processBodyError be this step: run fetchParams’s process
-  //    response end-of-body given response and failure.on.
+  //    response consume given response and failure.on.
   //    TODO
   //    3. If response’s body is null, then queue a fetch task to run
   //    processBody given null, with fetchParams’s task destination.on.
@@ -1256,11 +1275,15 @@ async function httpNetworkOrCacheFetch (
     // 3. If the ongoing fetch is terminated, then:
     if (context.terminated) {
       // 1. Let aborted be the termination’s aborted flag.
+      const aborted = context.terminated.aborted
+
       // 2. If aborted is set, then return an aborted network error.
+      if (aborted) {
+        return makeNetworkError(new AbortError())
+      }
+
       // 3. Return a network error.
-      return makeNetworkError(
-        context.terminated.aborted ? new AbortError() : null
-      )
+      return makeNetworkError(context.terminated.reason)
     }
 
     // 4. Prompt the end user as appropriate in request’s window and store
@@ -1290,10 +1313,12 @@ async function httpNetworkOrCacheFetch (
       const aborted = context.terminated.aborted
 
       // 2. If aborted is set, then return an aborted network error.
-      const reason = aborted ? new AbortError() : new Error('terminated')
+      if (aborted) {
+        return makeNetworkError(new AbortError())
+      }
 
       // 3. Return a network error.
-      return makeNetworkError(reason)
+      return makeNetworkError(context.terminated.reason)
     }
 
     // 2. Set response to the result of running HTTP-network-or-cache
@@ -1480,13 +1505,15 @@ function httpNetworkFetch (
       const aborted = this.terminated.aborted
 
       // 2. If connection uses HTTP/2, then transmit an RST_STREAM frame.
-      this.connection?.destroy()
+      this.connection.destroy()
 
       // 3. If aborted is set, then return an aborted network error.
-      const reason = aborted ? new AbortError() : new Error('terminated')
+      if (aborted) {
+        return resolve(makeNetworkError(new AbortError()))
+      }
 
       // 4. Return a network error.
-      resolve(makeNetworkError(reason))
+      return resolve(makeNetworkError(this.terminated.reason))
     }
 
     // 10. Let pullAlgorithm be an action that resumes the ongoing fetch
@@ -1531,7 +1558,6 @@ function httpNetworkFetch (
           await pullAlgorithm(controller)
         },
         async cancel (reason) {
-          stream[kError] = reason
           await cancelAlgorithm(reason)
         }
       },
@@ -1570,29 +1596,19 @@ function httpNetworkFetch (
         response.aborted = true
 
         // 2. If stream is readable, error stream with an "AbortError" DOMException.
-        try {
+        if (isReadable(stream)) {
           this.controller.error(new AbortError())
-        } catch (err) {
-          // Will throw TypeError if body is not readable.
-          if (err.name !== 'TypeError') {
-            throw err
-          }
         }
       } else {
         // 4. Otherwise, if stream is readable, error stream with a TypeError.
-        try {
+        if (isReadable(stream)) {
           this.controller.error(new TypeError('terminated'))
-        } catch (err) {
-          // Will throw TypeError if body is not readable.
-          if (err.name !== 'TypeError') {
-            throw err
-          }
         }
       }
 
       // 5. If connection uses HTTP/2, then transmit an RST_STREAM frame.
       // 6. Otherwise, the user agent should close connection unless it would be bad for performance to do so.
-      this.connection?.destroy()
+      this.connection.destroy()
     }
 
     // 19. Return response.
@@ -1638,8 +1654,6 @@ function httpNetworkFetch (
             )
           }
 
-          registry.register(stream, this.abort, this)
-
           response = makeResponse({
             status,
             statusText,
@@ -1672,29 +1686,16 @@ function httpNetworkFetch (
             }
           }
 
-          let iterator
-
           if (decoders.length > 1) {
-            if (compose) {
-              this.decoder = compose(...decoders)
-              iterator = this.decoder[Symbol.asyncIterator]()
-            } else {
-              this.decoder = new PassThrough()
-              iterator = pipeline(this.decoder, ...decoders, () => {})[
-                Symbol.asyncIterator
-              ]()
-            }
-          } else if (decoders.length === 1) {
-            this.decoder = decoders[0]
-            iterator = this.decoder[Symbol.asyncIterator]()
-          } else {
-            this.decoder = new PassThrough()
-            iterator = this.decoder[Symbol.asyncIterator]()
+            pipeline(...decoders, () => {})
+          } else if (decoders.length === 0) {
+            // TODO (perf): Avoid intermediate.
+            decoders.push(new PassThrough())
           }
 
-          if (this.decoder) {
-            this.decoder.on('drain', resume)
-          }
+          this.decoder = decoders[0].on('drain', resume)
+
+          const iterator = decoders[decoders.length - 1][Symbol.asyncIterator]()
 
           pullAlgorithm = async (controller) => {
             // 4. Set bytes to the result of handling content codings given
@@ -1738,8 +1739,8 @@ function httpNetworkFetch (
             controller.enqueue(new Uint8Array(bytes))
 
             // 8. If stream is errored, then terminate the ongoing fetch.
-            if (stream[kError]) {
-              this.context.terminate({ reason: stream[kError] })
+            if (isErrored(stream)) {
+              this.context.terminate()
               return
             }
 
@@ -1782,14 +1783,10 @@ function httpNetworkFetch (
         },
 
         onComplete () {
-          registry.unregister(this)
-
           this.decoder.end()
         },
 
         onError (error) {
-          registry.unregister(this)
-
           this.decoder?.destroy(error)
 
           this.context.terminate({ reason: error })

package/lib/fetch/request.js

@@ -28,6 +28,10 @@ let TransformStream
 
 const kInit = Symbol('init')
 
+const requestFinalizer = new FinalizationRegistry(({ signal, abort }) => {
+  signal.removeEventListener('abort', abort)
+})
+
 // https://fetch.spec.whatwg.org/#request-class
 class Request {
   // https://fetch.spec.whatwg.org/#dom-request
@@ -134,8 +138,48 @@ class Request {
 
     // 12. Set request to a new request with the following properties:
     request = makeRequest({
-      ...request,
-      window
+      // URL request’s URL.
+      // undici implementation note: this is set as the first item in request's urlList in makeRequest
+      // method request’s method. 
+      method: request.method,
+      // header list A copy of request’s header list.
+      // undici implementation note: headersList is cloned in makeRequest
+      headersList: request.headersList,
+      // unsafe-request flag Set. 
+      unsafeRequest: request.unsafeRequest,
+      // client This’s relevant settings object.
+      client: request.client,
+      // window window.
+      window,
+      // priority request’s priority. 
+      priority: request.priority,
+      // origin request’s origin. The propagation of the origin is only significant for navigation requests
+      // being handled by a service worker. In this scenario a request can have an origin that is different 
+      // from the current client.
+      origin: request.origin,
+      // referrer request’s referrer.
+      referrer: request.referrer, 
+      // referrer policy request’s referrer policy.
+      referrerPolicy: request.referrerPolicy,
+      // mode request’s mode.
+      mode: request.mode, 
+      // credentials mode request’s credentials mode.
+      credentials: request.credentials,
+      // cache mode request’s cache mode.
+      cache: request.cache,
+      // redirect mode request’s redirect mode.
+      redirect: request.redirect, 
+      // integrity metadata request’s integrity metadata.
+      integrity: request.integrity,
+      // keepalive request’s keepalive.
+      keepalive: request.keepalive,
+      // reload-navigation flag request’s reload-navigation flag.
+      reloadNavigation: request.reloadNavigation,
+      // history-navigation flag request’s history-navigation flag.
+      historyNavigation: request.historyNavigation,
+      // URL list A clone of request’s URL list.
+      // undici implementation note: urlList is cloned in makeRequest
+      urlList: request.urlList
     })
 
     // 13. If init is not empty, then:
@@ -151,11 +195,20 @@ class Request {
       // 3. Unset request’s history-navigation flag.
       request.historyNavigation = false
 
-      // 4. Set request’s referrer to "client"
+      // 4. Set request’s origin to "client".
+      request.origin = 'client'
+
+      // 5. Set request’s referrer to "client"
       request.referrer = 'client'
 
-      // 5. Set request’s referrer policy to the empty string.
+      // 6. Set request’s referrer policy to the empty string.
       request.referrerPolicy = ''
+
+      // 7. Set request’s URL to request’s current URL.
+      request.url = request.urlList[request.urlList.length - 1]
+
+      // 8. Set request’s URL list to « request’s URL ».
+      request.urlList = [request.url]
     }
 
     // 14. If init["referrer"] exists, then:
@@ -164,7 +217,7 @@ class Request {
       const referrer = init.referrer
 
       // 2. If referrer is the empty string, then set request’s referrer to "no-referrer".
-      if (!referrer === '') {
+      if (referrer === '') {
         request.referrer = 'no-referrer'
       } else {
         // 1. Let parsedReferrer be the result of parsing referrer with
@@ -328,14 +381,9 @@ class Request {
       if (signal.aborted) {
         ac.abort()
       } else {
-        // TODO: Remove this listener on failure/success.
-        signal.addEventListener(
-          'abort',
-          function () {
-            ac.abort()
-          },
-          { once: true }
-        )
+        const abort = () => ac.abort()
+        signal.addEventListener('abort', abort, { once: true })
+        requestFinalizer.register(this, { signal, abort })
       }
     }
 

package/lib/fetch/symbols.js

@@ -6,6 +6,5 @@ module.exports = {
   kSignal: Symbol('signal'),
   kState: Symbol('state'),
   kGuard: Symbol('guard'),
-  kRealm: Symbol('realm'),
-  kError: Symbol('error')
+  kRealm: Symbol('realm')
 }

package/lib/fetch/util.js

@@ -253,20 +253,20 @@ function coarsenedSharedCurrentTime (crossOriginIsolatedCapability) {
   return performance.now()
 }
 
-function makeTimingInfo (init) {
+// https://fetch.spec.whatwg.org/#create-an-opaque-timing-info
+function createOpaqueTimingInfo (timingInfo) {
   return {
-    startTime: 0,
+    startTime: timingInfo.startTime ?? 0,
     redirectStartTime: 0,
     redirectEndTime: 0,
-    postRedirectStartTime: 0,
+    postRedirectStartTime: timingInfo.startTime ?? 0,
     finalServiceWorkerStartTime: 0,
     finalNetworkResponseStartTime: 0,
     finalNetworkRequestStartTime: 0,
     endTime: 0,
     encodedBodySize: 0,
     decodedBodySize: 0,
-    finalConnectionTimingInfo: null,
-    ...init
+    finalConnectionTimingInfo: null
   }
 }
 
@@ -318,7 +318,7 @@ module.exports = {
   TAOCheck,
   corsCheck,
   crossOriginResourcePolicyCheck,
-  makeTimingInfo,
+  createOpaqueTimingInfo,
   setRequestReferrerPolicyOnRedirect,
   isValidHTTPToken,
   requestBadPort,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "undici",
-  "version": "4.11.3",
+  "version": "4.12.0",
   "description": "An HTTP/1.1 client, written from scratch for Node.js",
   "homepage": "https://undici.nodejs.org",
   "bugs": {