@podium/client 5.2.4 → 5.2.5-beta.2

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+## [5.2.5-beta.2](https://github.com/podium-lib/client/compare/v5.2.5-beta.1...v5.2.5-beta.2) (2025-01-13)
+
+
+### Bug Fixes
+
+* update @podium/http-client to version 1.0.0-beta.9 ([e64596a](https://github.com/podium-lib/client/commit/e64596a8bf194ecee5702f2dda61674c9a5f8772))
+
+## [5.2.5-beta.1](https://github.com/podium-lib/client/compare/v5.2.4...v5.2.5-beta.1) (2024-12-02)
+
+
+### Bug Fixes
+
+* adding @podium/http-client beta ([e300bde](https://github.com/podium-lib/client/commit/e300bde2fd8e6356227e03a8d9ae847e509cd1f4))
+
 ## [5.2.4](https://github.com/podium-lib/client/compare/v5.2.3...v5.2.4) (2024-11-21)
 
 

package/lib/resolver.content.js

@@ -6,9 +6,9 @@ import { Boom, badGateway } from '@hapi/boom';
 import { join, dirname } from 'path';
 import { fileURLToPath } from 'url';
 import fs from 'fs';
+import HttpClient from '@podium/http-client';
 import * as utils from './utils.js';
 import Response from './response.js';
-import HTTP from './http.js';
 import { parseLinkHeaders, filterAssets } from './utils.js';
 import { AssetJs, AssetCss } from '@podium/utils';
 
@@ -25,7 +25,7 @@ const UA_STRING = `${pkg.name} ${pkg.version}`;
 /**
  * @typedef {object} PodletClientContentResolverOptions
  * @property {string} clientName
- * @property {import('./http.js').default} [http]
+ * @property {import('@podium/http-client').default} [http]
  * @property {import('abslog').AbstractLoggerOptions} [logger]
  */
 
@@ -41,9 +41,9 @@ export default class PodletClientContentResolver {
      */
     // @ts-expect-error Deliberate default empty options for better error messages
     constructor(options = {}) {
-        this.#http = options.http || new HTTP();
         const name = options.clientName;
         this.#log = abslog(options.logger);
+        this.#http = options.http || new HttpClient({ logger: options.logger });
         this.#metrics = new Metrics();
         this.#histogram = this.#metrics.histogram({
             name: 'podium_client_resolver_content_resolve',
@@ -62,6 +62,7 @@ export default class PodletClientContentResolver {
                 error,
             );
         });
+        this.#http.metrics.pipe(this.#metrics);
     }
 
     get metrics() {
@@ -153,11 +154,16 @@ export default class PodletClientContentResolver {
         );
 
         try {
+            const url = new URL(uri);
             const {
                 statusCode,
                 headers: hdrs,
                 body,
-            } = await this.#http.request(uri, reqOptions);
+            } = await this.#http.request({
+                origin: url.origin,
+                path: url.pathname,
+                ...reqOptions,
+            });
 
             const parsedAssetObjects = parseLinkHeaders(hdrs.link);
 
@@ -252,7 +258,6 @@ export default class PodletClientContentResolver {
             if (outgoing.redirectable && statusCode >= 300) {
                 outgoing.redirect = {
                     statusCode,
-                    // @ts-expect-error TODO: look into what happens if the podlet returns more than one location header
                     location: hdrs && hdrs.location,
                 };
             }

package/lib/resolver.fallback.js

@@ -3,9 +3,9 @@ import Metrics from '@metrics/client';
 import { join, dirname } from 'path';
 import { fileURLToPath } from 'url';
 import fs from 'fs';
-import HTTP from './http.js';
 import { parseLinkHeaders, filterAssets } from './utils.js';
 import { AssetJs, AssetCss } from '@podium/utils';
+import HttpClient from '@podium/http-client';
 
 const currentDirectory = dirname(fileURLToPath(import.meta.url));
 
@@ -20,7 +20,7 @@ const UA_STRING = `${pkg.name} ${pkg.version}`;
 /**
  * @typedef {object} PodletClientFallbackResolverOptions
  * @property {string} clientName
- * @property {import('./http.js').default} [http]
+ * @property {import('@podium/http-client').default} [http]
  * @property {import('abslog').AbstractLoggerOptions} [logger]
  */
 
@@ -36,9 +36,9 @@ export default class PodletClientFallbackResolver {
      */
     // @ts-expect-error Deliberate default empty options for better error messages
     constructor(options = {}) {
-        this.#http = options.http || new HTTP();
         const name = options.clientName;
         this.#log = abslog(options.logger);
+        this.#http = options.http || new HttpClient({ logger: options.logger });
         this.#metrics = new Metrics();
         this.#histogram = this.#metrics.histogram({
             name: 'podium_client_resolver_fallback_resolve',
@@ -57,6 +57,7 @@ export default class PodletClientFallbackResolver {
                 error,
             );
         });
+        this.#http.metrics.pipe(this.#metrics);
     }
 
     get metrics() {
@@ -114,11 +115,16 @@ export default class PodletClientFallbackResolver {
             this.#log.debug(
                 `start reading fallback content from remote resource - resource: ${outgoing.name} - url: ${outgoing.fallbackUri}`,
             );
+            const url = new URL(outgoing.fallbackUri);
             const {
                 statusCode,
                 body,
                 headers: resHeaders,
-            } = await this.#http.request(outgoing.fallbackUri, reqOptions);
+            } = await this.#http.request({
+                origin: url.origin,
+                path: url.pathname,
+                ...reqOptions,
+            });
 
             const parsedAssetObjects = parseLinkHeaders(resHeaders.link);
 

package/lib/resolver.js

@@ -1,7 +1,7 @@
 import Metrics from '@metrics/client';
 import abslog from 'abslog';
 import assert from 'assert';
-import HTTP from './http.js';
+import HttpClient from '@podium/http-client';
 import Manifest from './resolver.manifest.js';
 import Fallback from './resolver.fallback.js';
 import Content from './resolver.content.js';
@@ -34,18 +34,18 @@ export default class PodletClientResolver {
         );
 
         const log = abslog(options.logger);
-        const http = new HTTP();
+        const httpClient = new HttpClient({
+            clientName: options.clientName,
+            logger: options.logger,
+        });
         this.#cache = new Cache(registry, options);
         this.#manifest = new Manifest({
             clientName: options.clientName,
             logger: options.logger,
-            http,
-        });
-        this.#fallback = new Fallback({ ...options, http });
-        this.#content = new Content({
-            ...options,
-            http,
+            http: httpClient,
         });
+        this.#fallback = new Fallback({ ...options, http: httpClient });
+        this.#content = new Content({ ...options, http: httpClient });
         this.#metrics = new Metrics();
 
         this.#metrics.on('error', (error) => {
@@ -55,6 +55,7 @@ export default class PodletClientResolver {
             );
         });
 
+        httpClient.metrics.pipe(this.#metrics);
         this.#content.metrics.pipe(this.#metrics);
         this.#fallback.metrics.pipe(this.#metrics);
         this.#manifest.metrics.pipe(this.#metrics);

package/lib/resolver.manifest.js

@@ -6,7 +6,7 @@ import * as utils from '@podium/utils';
 import { join, dirname } from 'path';
 import { fileURLToPath } from 'url';
 import fs from 'fs';
-import HTTP from './http.js';
+import HttpClient from '@podium/http-client';
 
 const currentDirectory = dirname(fileURLToPath(import.meta.url));
 
@@ -22,7 +22,7 @@ const UA_STRING = `${pkg.name} ${pkg.version}`;
  * @typedef {object} PodletClientManifestResolverOptions
  * @property {string} clientName
  * @property {import('abslog').AbstractLoggerOptions} [logger]
- * @property {import('./http.js').default} [http]
+ * @property {import('@podium/http-client').default} [http]
  */
 
 export default class PodletClientManifestResolver {
@@ -37,9 +37,9 @@ export default class PodletClientManifestResolver {
      */
     // @ts-expect-error Deliberate default empty options for better error messages
     constructor(options = {}) {
-        this.#http = options.http || new HTTP();
         const name = options.clientName;
         this.#log = abslog(options.logger);
+        this.#http = options.http || new HttpClient({ logger: this.#log });
         this.#metrics = new Metrics();
         this.#histogram = this.#metrics.histogram({
             name: 'podium_client_resolver_manifest_resolve',
@@ -58,6 +58,7 @@ export default class PodletClientManifestResolver {
                 error,
             );
         });
+        this.#http.metrics.pipe(this.#metrics);
     }
 
     get metrics() {
@@ -97,10 +98,12 @@ export default class PodletClientManifestResolver {
         );
 
         try {
-            const response = await this.#http.request(
-                outgoing.manifestUri,
-                reqOptions,
-            );
+            const url = new URL(outgoing.manifestUri);
+            const response = await this.#http.request({
+                origin: url.origin,
+                path: url.pathname,
+                ...reqOptions,
+            });
             const { statusCode, headers: hdrs, body } = response;
 
             // Remote responds but with an http error code
@@ -120,11 +123,10 @@ export default class PodletClientManifestResolver {
                 await body.text();
                 return outgoing;
             }
-
+            // Body must be consumed; https://github.com/nodejs/undici/issues/583#issuecomment-855384858
+            const schema = await body.json();
             const manifest = validateManifest(
-                /** @type {import("@podium/schemas").PodletManifestSchema} */ (
-                    await body.json()
-                ),
+                /** @type {import("@podium/schemas").PodletManifestSchema} */ schema,
             );
 
             // Manifest validation error

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@podium/client",
-  "version": "5.2.4",
+  "version": "5.2.5-beta.2",
   "type": "module",
   "license": "MIT",
   "keywords": [
@@ -39,11 +39,13 @@
   "dependencies": {
     "@hapi/boom": "10.0.1",
     "@metrics/client": "2.5.4",
+    "@podium/http-client": "1.0.0-beta.9",
     "@podium/schemas": "5.1.0",
     "@podium/utils": "5.4.0",
     "abslog": "2.4.4",
     "http-cache-semantics": "^4.0.3",
     "lodash.clonedeep": "^4.5.0",
+    "opossum": "^8.3.0",
     "ttl-mem-cache": "4.1.0",
     "undici": "6.21.0"
   },

package/types/resolver.content.d.ts

@@ -1,7 +1,7 @@
 /**
  * @typedef {object} PodletClientContentResolverOptions
  * @property {string} clientName
- * @property {import('./http.js').default} [http]
+ * @property {import('@podium/http-client').default} [http]
  * @property {import('abslog').AbstractLoggerOptions} [logger]
  */
 export default class PodletClientContentResolver {
@@ -23,7 +23,7 @@ export default class PodletClientContentResolver {
 }
 export type PodletClientContentResolverOptions = {
     clientName: string;
-    http?: import("./http.js").default;
+    http?: any;
     logger?: import("abslog").AbstractLoggerOptions;
 };
 import Metrics from '@metrics/client';

package/types/resolver.fallback.d.ts

@@ -1,7 +1,7 @@
 /**
  * @typedef {object} PodletClientFallbackResolverOptions
  * @property {string} clientName
- * @property {import('./http.js').default} [http]
+ * @property {import('@podium/http-client').default} [http]
  * @property {import('abslog').AbstractLoggerOptions} [logger]
  */
 export default class PodletClientFallbackResolver {
@@ -23,7 +23,7 @@ export default class PodletClientFallbackResolver {
 }
 export type PodletClientFallbackResolverOptions = {
     clientName: string;
-    http?: import("./http.js").default;
+    http?: any;
     logger?: import("abslog").AbstractLoggerOptions;
 };
 import Metrics from '@metrics/client';

package/types/resolver.manifest.d.ts

@@ -2,7 +2,7 @@
  * @typedef {object} PodletClientManifestResolverOptions
  * @property {string} clientName
  * @property {import('abslog').AbstractLoggerOptions} [logger]
- * @property {import('./http.js').default} [http]
+ * @property {import('@podium/http-client').default} [http]
  */
 export default class PodletClientManifestResolver {
     /**
@@ -22,6 +22,6 @@ export default class PodletClientManifestResolver {
 export type PodletClientManifestResolverOptions = {
     clientName: string;
     logger?: import("abslog").AbstractLoggerOptions;
-    http?: import("./http.js").default;
+    http?: any;
 };
 import Metrics from '@metrics/client';