@podium/client 5.1.10-next.2 → 5.1.10

package/CHANGELOG.md

@@ -1,16 +1,9 @@
-## [5.1.10-next.2](https://github.com/podium-lib/client/compare/v5.1.10-next.1...v5.1.10-next.2) (2024-08-26)
+## [5.1.10](https://github.com/podium-lib/client/compare/v5.1.9...v5.1.10) (2024-09-06)
 
 
 ### Bug Fixes
 
-* use AbortController instead of AbortSignal to avoid unhandled exception ([#412](https://github.com/podium-lib/client/issues/412)) ([87f5ffe](https://github.com/podium-lib/client/commit/87f5ffe553aa49189658a9be0e19d1323878a55a))
-
-## [5.1.10-next.1](https://github.com/podium-lib/client/compare/v5.1.9...v5.1.10-next.1) (2024-08-22)
-
-
-### Bug Fixes
-
-* use AbortSignal to ensure timeouts are respected ([08899d9](https://github.com/podium-lib/client/commit/08899d974246037cb1893a5b6d06bd6df58815e2))
+* **deps:** update dependency @podium/utils to v5.2.0 ([f7d4675](https://github.com/podium-lib/client/commit/f7d4675e6e2b9f2fba26cbd164d3e7a73b9c132e))
 
 ## [5.1.9](https://github.com/podium-lib/client/compare/v5.1.8...v5.1.9) (2024-08-19)
 

package/lib/http.js

@@ -1,4 +1,4 @@
-import { request as undiciRequest } from 'undici';
+import { request } from 'undici';
 
 /**
  * @typedef {object} PodiumHttpClientRequestOptions
@@ -7,38 +7,22 @@ import { request as undiciRequest } from 'undici';
  * @property {boolean} [rejectUnauthorized]
  * @property {boolean} [follow]
  * @property {number} [timeout]
+ * @property {number} [bodyTimeout]
  * @property {object} [query]
  * @property {import('http').IncomingHttpHeaders} [headers]
  */
 
 export default class HTTP {
-    constructor(requestFn = undiciRequest) {
-        this.requestFn = requestFn;
-    }
-
     /**
      * @param {string} url
      * @param {PodiumHttpClientRequestOptions} options
      * @returns {Promise<Pick<import('undici').Dispatcher.ResponseData, 'statusCode' | 'headers' | 'body'>>}
      */
     async request(url, options) {
-        const abortController = new AbortController();
-
-        const timeoutId = setTimeout(() => {
-            abortController.abort();
-        }, options.timeout || 1000);
-
-        try {
-            const { statusCode, headers, body } = await this.requestFn(
-                new URL(url),
-                {
-                    ...options,
-                    signal: abortController.signal,
-                },
-            );
-            return { statusCode, headers, body };
-        } finally {
-            clearTimeout(timeoutId);
-        }
+        const { statusCode, headers, body } = await request(
+            new URL(url),
+            options,
+        );
+        return { statusCode, headers, body };
     }
 }

package/lib/resolver.content.js

@@ -136,7 +136,7 @@ export default class PodletClientContentResolver {
         /** @type {import('./http.js').PodiumHttpClientRequestOptions} */
         const reqOptions = {
             rejectUnauthorized: outgoing.rejectUnauthorized,
-            timeout: outgoing.timeout,
+            bodyTimeout: outgoing.timeout,
             method: 'GET',
             query: outgoing.reqOptions.query,
             headers,
@@ -267,7 +267,6 @@ export default class PodletClientContentResolver {
                 }),
             );
 
-            // @ts-ignore
             pipeline([body, outgoing], (err) => {
                 if (err) {
                     this.#log.warn('error while piping content stream', err);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@podium/client",
-  "version": "5.1.10-next.2",
+  "version": "5.1.10",
   "type": "module",
   "license": "MIT",
   "keywords": [
@@ -40,7 +40,7 @@
     "@hapi/boom": "10.0.1",
     "@metrics/client": "2.5.3",
     "@podium/schemas": "5.0.6",
-    "@podium/utils": "5.1.0",
+    "@podium/utils": "5.2.0",
     "abslog": "2.4.4",
     "http-cache-semantics": "^4.0.3",
     "lodash.clonedeep": "^4.5.0",
@@ -51,10 +51,10 @@
     "@podium/test-utils": "2.5.2",
     "@semantic-release/changelog": "6.0.3",
     "@semantic-release/git": "10.0.1",
-    "@semantic-release/github": "10.0.6",
+    "@semantic-release/github": "10.1.7",
     "@semantic-release/npm": "12.0.1",
     "@semantic-release/release-notes-generator": "13.0.0",
-    "@sinonjs/fake-timers": "11.2.2",
+    "@sinonjs/fake-timers": "11.3.1",
     "@types/readable-stream": "4.0.15",
     "benchmark": "2.1.4",
     "eslint": "9.6.0",
@@ -62,10 +62,10 @@
     "eslint-plugin-prettier": "5.1.3",
     "express": "4.19.2",
     "get-stream": "9.0.1",
-    "globals": "15.8.0",
+    "globals": "15.9.0",
     "http-proxy": "1.18.1",
     "is-stream": "4.0.1",
-    "npm-run-all2": "5.0.0",
+    "npm-run-all2": "5.0.2",
     "prettier": "3.3.2",
     "semantic-release": "23.1.1",
     "tap": "18.7.2",

package/types/http.d.ts

@@ -5,12 +5,11 @@
  * @property {boolean} [rejectUnauthorized]
  * @property {boolean} [follow]
  * @property {number} [timeout]
+ * @property {number} [bodyTimeout]
  * @property {object} [query]
  * @property {import('http').IncomingHttpHeaders} [headers]
  */
 export default class HTTP {
-    constructor(requestFn?: typeof undiciRequest);
-    requestFn: typeof undiciRequest;
     /**
      * @param {string} url
      * @param {PodiumHttpClientRequestOptions} options
@@ -24,7 +23,7 @@ export type PodiumHttpClientRequestOptions = {
     rejectUnauthorized?: boolean;
     follow?: boolean;
     timeout?: number;
+    bodyTimeout?: number;
     query?: object;
     headers?: import('http').IncomingHttpHeaders;
 };
-import { request as undiciRequest } from 'undici';