undici 4.9.2 → 4.9.3

package/docs/api/BalancedPool.md

@@ -25,10 +25,6 @@ See: [`PoolOptions`](docs/api/Pool.md#parameter-pooloptions)
 
 Returns an array of upstreams that were previously added.
 
-### `BalancedPool.busy`
-
-Implements [Client.busy](docs/api/Client.md#clientbusy)
-
 ### `BalancedPool.closed`
 
 Implements [Client.closed](docs/api/Client.md#clientclosed)

package/docs/api/Client.md

@@ -113,12 +113,6 @@ See [`Dispatcher.upgrade(options[, callback])`](docs/api/Dispatcher.md#clientupg
 
 ## Instance Properties
 
-### `Client.busy`
-
-* `boolean`
-
-`true` if pipeline is saturated or blocked. Indicates whether dispatching further requests is meaningful.
-
 ### `Client.closed`
 
 * `boolean`
@@ -161,7 +155,7 @@ const server = createServer((request, response) => {
   response.end('Hello, World!')
 }).listen()
 
-await once(server, 'listening') 
+await once(server, 'listening')
 
 const client = new Client(`http://localhost:${server.address().port}`)
 
@@ -251,7 +245,6 @@ const client = new Client(`http://localhost:${server.address().port}`)
 
 client.on('drain', () => {
   console.log('drain event')
-  console.log(`Is Client busy: ${client.busy}`)
   client.close()
   server.close()
 })
@@ -262,8 +255,6 @@ const requests = [
   client.request({ path: '/', method: 'GET' })
 ]
 
-console.log(`Is Client busy: ${client.busy}`)
-
 await Promise.all(requests)
 
 console.log('requests completed')

package/docs/api/Dispatcher.md

@@ -177,14 +177,15 @@ try {
 ### `Dispatcher.dispatch(options, handler)`
 
 This is the low level API which all the preceding APIs are implemented on top of.
-This API is expected to evolve through semver-major versions and is less stable than the preceding higher level APIs. It is primarily intended for library developers who implement higher level APIs on top of this.
+This API is expected to evolve through semver-major versions and is less stable than the preceding higher level APIs.
+It is primarily intended for library developers who implement higher level APIs on top of this.
 
 Arguments:
 
 * **options** `DispatchOptions`
 * **handler** `DispatchHandler`
 
-Returns: `void`
+Returns: `Boolean` - `false` if dispatcher is busy and further dispatch calls won't make any progress until the `'drain'` event has been emitted.
 
 #### Parameter: `DispatchOptions`
 
@@ -330,7 +331,7 @@ Extends: [`RequestOptions`](#parameter-requestoptions)
 * **opaque** `unknown`
 * **body** `stream.Readable`
 * **context** `object`
-* **onInfo** `({statusCode: number, headers: Record<string, string | string[]>}) => void | null` (optional) - Default: `null` - Callback collecting all the info headers (HTTP 100-199) received. 
+* **onInfo** `({statusCode: number, headers: Record<string, string | string[]>}) => void | null` (optional) - Default: `null` - Callback collecting all the info headers (HTTP 100-199) received.
 
 #### Example 1 - Pipeline Echo
 
@@ -413,7 +414,7 @@ Extends: [`DispatchOptions`](#parameter-dispatchoptions)
 
 * **opaque** `unknown` (optional) - Default: `null` - Used for passing through context to `ResponseData`
 * **signal** `AbortSignal | events.EventEmitter | null` (optional) - Default: `null`
-* **onInfo** `({statusCode: number, headers: Record<string, string | string[]>}) => void | null` (optional) - Default: `null` - Callback collecting all the info headers (HTTP 100-199) received. 
+* **onInfo** `({statusCode: number, headers: Record<string, string | string[]>}) => void | null` (optional) - Default: `null` - Callback collecting all the info headers (HTTP 100-199) received.
 
 The `RequestOptions.method` property should not be value `'CONNECT'`.
 
@@ -581,7 +582,7 @@ Returns: `void | Promise<StreamData>` - Only returns a `Promise` if no `callback
 * **statusCode** `number`
 * **headers** `http.IncomingHttpHeaders`
 * **opaque** `unknown`
-* **onInfo** `({statusCode: number, headers: Record<string, string | string[]>}) => void | null` (optional) - Default: `null` - Callback collecting all the info headers (HTTP 100-199) received. 
+* **onInfo** `({statusCode: number, headers: Record<string, string | string[]>}) => void | null` (optional) - Default: `null` - Callback collecting all the info headers (HTTP 100-199) received.
 
 #### Parameter: `StreamData`
 

package/docs/api/Pool.md

@@ -22,10 +22,6 @@ Extends: [`ClientOptions`](docs/api/Client.md#parameter-clientoptions)
 
 ## Instance Properties
 
-### `Pool.busy`
-
-Implements [Client.busy](docs/api/Client.md#clientbusy)
-
 ### `Pool.closed`
 
 Implements [Client.closed](docs/api/Client.md#clientclosed)

package/docs/api/api-lifecycle.md

@@ -59,4 +59,4 @@ This sub-state is only entered when a `Client` instance has queued requests and
 
 ### destroyed
 
-The **destroyed** state is a final state for the `Client` instance. Once in this state, a `Client` is nonfunctional. Calling any other `Client` methods will result in an `ClientDestroyedError`.
+The **destroyed** state is a final state for the `Client` instance. Once in this state, a `Client` is nonfunctional. Calling any other `Client` methods will result in an `ClientDestroyedError`.

package/lib/balanced-pool.js

@@ -44,6 +44,8 @@ class BalancedPool extends Dispatcher {
     })
 
     pool.on('drain', (...args) => {
+      pool[kNeedDrain] = false
+
       if (this[kNeedDrain]) {
         this[kNeedDrain] = false
         this.emit('drain', ...args)
@@ -62,12 +64,16 @@ class BalancedPool extends Dispatcher {
       throw new BalancedPoolMissingUpstreamError()
     }
 
-    let pool = this[kPools].shift()
-    this[kPools].push(pool)
-    if (pool.busy) {
-      pool = this[kPools].find(pool => !pool.busy) || pool
+    const pool = this[kPools].find(pool => !pool[kNeedDrain]) || this[kPools][0]
+
+    if (!pool.dispatch(opts, handler)) {
+      pool[kNeedDrain] = true
+      this[kNeedDrain] = true
     }
-    this[kNeedDrain] = !pool.dispatch(opts, handler)
+
+    this[kPools].splice(this[kPools].indexOf(pool), 1)
+    this[kPools].push(pool)
+
     return !this[kNeedDrain]
   }
 
@@ -87,10 +93,6 @@ class BalancedPool extends Dispatcher {
     return this[kPools].reduce((acc, pool) => acc && pool.destroyed, true)
   }
 
-  get busy () {
-    return this[kPools].reduce((acc, pool) => acc && pool.busy, true) || false
-  }
-
   get closed () {
     return this[kPools].reduce((acc, pool) => acc && pool.closed, true)
   }

package/lib/fetch/headers.js

@@ -313,20 +313,23 @@ class Headers {
   }
 
   * keys () {
-    for (let index = 0; index < this[kHeadersList].length; index += 2) {
-      yield this[kHeadersList][index]
+    const clone = this[kHeadersList].slice()
+    for (let index = 0; index < clone.length; index += 2) {
+      yield clone[index]
     }
   }
 
   * values () {
-    for (let index = 1; index < this[kHeadersList].length; index += 2) {
-      yield this[kHeadersList][index]
+    const clone = this[kHeadersList].slice()
+    for (let index = 1; index < clone.length; index += 2) {
+      yield clone[index]
     }
   }
 
   * entries () {
-    for (let index = 0; index < this[kHeadersList].length; index += 2) {
-      yield [this[kHeadersList][index], this[kHeadersList][index + 1]]
+    const clone = this[kHeadersList].slice()
+    for (let index = 0; index < clone.length; index += 2) {
+      yield [clone[index], clone[index + 1]]
     }
   }
 

package/package.json

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

package/types/agent.d.ts

@@ -11,7 +11,7 @@ declare class Agent extends Dispatcher{
   /** `true` after `dispatcher.destroyed()` has been called or `dispatcher.close()` has been called and the dispatcher shutdown has completed. */
   destroyed: boolean;
   /** Dispatches a request. */
-  dispatch(options: Agent.DispatchOptions, handler: Dispatcher.DispatchHandlers): void;
+  dispatch(options: Agent.DispatchOptions, handler: Dispatcher.DispatchHandlers): boolean;
 }
 
 declare namespace Agent {

package/types/dispatcher.d.ts

@@ -11,7 +11,7 @@ export = Dispatcher;
 /** Dispatcher is the core API used to dispatch requests. */
 declare class Dispatcher extends EventEmitter {
   /** Dispatches a request. This API is expected to evolve through semver-major versions and is less stable than the preceding higher level APIs. It is primarily intended for library developers who implement higher level APIs on top of this. */
-  dispatch(options: Dispatcher.DispatchOptions, handler: Dispatcher.DispatchHandlers): void;
+  dispatch(options: Dispatcher.DispatchOptions, handler: Dispatcher.DispatchHandlers): boolean;
   /** Starts two-way communications with the requested resource. */
   connect(options: Dispatcher.ConnectOptions): Promise<Dispatcher.ConnectData>;
   connect(options: Dispatcher.ConnectOptions, callback: (err: Error | null, data: Dispatcher.ConnectData) => void): void;
@@ -148,7 +148,7 @@ declare namespace Dispatcher {
   export type PipelineHandler = (data: PipelineHandlerData) => Readable;
   export type HttpMethod = 'GET' | 'HEAD' | 'POST' | 'PUT' | 'DELETE' | 'CONNECT' | 'OPTIONS' | 'TRACE' | 'PATCH';
 
-  /** 
+  /**
    * @link https://fetch.spec.whatwg.org/#body-mixin
    */
   interface BodyMixin {

package/types/mock-agent.d.ts

@@ -12,7 +12,7 @@ declare class MockAgent<TMockAgentOptions extends MockAgent.Options = MockAgent.
   get<TInterceptable extends Interceptable>(origin: RegExp): TInterceptable;
   get<TInterceptable extends Interceptable>(origin: ((origin: string) => boolean)): TInterceptable;
   /** Dispatches a mocked request. */
-  dispatch(options: Agent.DispatchOptions, handler: Dispatcher.DispatchHandlers): void;
+  dispatch(options: Agent.DispatchOptions, handler: Dispatcher.DispatchHandlers): boolean;
   /** Closes the mock agent and waits for registered mock pools and clients to also close before resolving. */
   close(): Promise<void>;
   /** Disables mocking in MockAgent. */

package/types/mock-client.d.ts

@@ -11,7 +11,7 @@ declare class MockClient extends Client implements Interceptable {
   /** Intercepts any matching requests that use the same origin as this mock client. */
   intercept(options: MockInterceptor.Options): MockInterceptor;
   /** Dispatches a mocked request. */
-  dispatch(options: Dispatcher.DispatchOptions, handlers: Dispatcher.DispatchHandlers): void;
+  dispatch(options: Dispatcher.DispatchOptions, handlers: Dispatcher.DispatchHandlers): boolean;
   /** Closes the mock client and gracefully waits for enqueued requests to complete. */
   close(): Promise<void>;
 }

package/types/mock-pool.d.ts

@@ -11,7 +11,7 @@ declare class MockPool extends Pool implements Interceptable {
   /** Intercepts any matching requests that use the same origin as this mock pool. */
   intercept(options: MockInterceptor.Options): MockInterceptor;
   /** Dispatches a mocked request. */
-  dispatch(options: Dispatcher.DispatchOptions, handlers: Dispatcher.DispatchHandlers): void;
+  dispatch(options: Dispatcher.DispatchOptions, handlers: Dispatcher.DispatchHandlers): boolean;
   /** Closes the mock pool and gracefully waits for enqueued requests to complete. */
   close(): Promise<void>;
 }

package/types/proxy-agent.d.ts

@@ -6,7 +6,7 @@ export = ProxyAgent
 declare class ProxyAgent extends Dispatcher {
   constructor(options: ProxyAgent.Options | string)
 
-  dispatch(options: Agent.DispatchOptions, handler: Dispatcher.DispatchHandlers): void;
+  dispatch(options: Agent.DispatchOptions, handler: Dispatcher.DispatchHandlers): boolean;
   close(): Promise<void>;
 }