@podium/client 5.2.3 → 5.2.5

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+## [5.2.5](https://github.com/podium-lib/client/compare/v5.2.4...v5.2.5) (2024-11-27)
+
+
+### Bug Fixes
+
+* correctly build asset urls from link headers ([#446](https://github.com/podium-lib/client/issues/446)) ([5a30079](https://github.com/podium-lib/client/commit/5a30079fb47ee15a49ae6ff77348c400df3fe99a))
+
+## [5.2.4](https://github.com/podium-lib/client/compare/v5.2.3...v5.2.4) (2024-11-21)
+
+
+### Bug Fixes
+
+* reducing logl evel to info on 404 errors ([e6d6d04](https://github.com/podium-lib/client/commit/e6d6d04a4b30729691cafa68521bcfc743c3db64))
+
 ## [5.2.3](https://github.com/podium-lib/client/compare/v5.2.2...v5.2.3) (2024-11-13)
 
 

package/lib/resolver.content.js

@@ -159,7 +159,10 @@ export default class PodletClientContentResolver {
                 body,
             } = await this.#http.request(uri, reqOptions);
 
-            const parsedAssetObjects = parseLinkHeaders(hdrs.link);
+            const parsedAssetObjects = parseLinkHeaders(
+                hdrs.link,
+                outgoing.manifestUri,
+            );
 
             const scriptObjects = parsedAssetObjects.filter(
                 (asset) => asset instanceof AssetJs,
@@ -205,7 +208,7 @@ export default class PodletClientContentResolver {
                     },
                 });
 
-                this.#log.warn(
+                this.#log.debug(
                     `remote resource responded with non 200 http status code for content - code: ${statusCode} - resource: ${outgoing.name} - url: ${uri}`,
                 );
                 outgoing.success = true;

package/lib/resolver.fallback.js

@@ -120,7 +120,10 @@ export default class PodletClientFallbackResolver {
                 headers: resHeaders,
             } = await this.#http.request(outgoing.fallbackUri, reqOptions);
 
-            const parsedAssetObjects = parseLinkHeaders(resHeaders.link);
+            const parsedAssetObjects = parseLinkHeaders(
+                resHeaders.link,
+                outgoing.manifestUri,
+            );
 
             const scriptObjects = parsedAssetObjects.filter(
                 (asset) => asset instanceof AssetJs,

package/lib/utils.js

@@ -1,4 +1,4 @@
-import { AssetJs, AssetCss } from '@podium/utils';
+import { AssetJs, AssetCss, uriRelativeToAbsolute } from '@podium/utils';
 
 /**
  * Checks if a header object has a header.
@@ -75,14 +75,17 @@ export const filterAssets = (scope, assets) => {
 };
 
 // parse link headers in AssetCss and AssetJs objects
-export const parseLinkHeaders = (headers) => {
+export const parseLinkHeaders = (headers, manifestUri) => {
     const links = [];
 
     if (!headers) return links;
     headers.split(',').forEach((link) => {
         const parts = link.split(';');
         const [href, ...rest] = parts;
-        const value = href.replace(/<|>|"/g, '').trim();
+        const value = uriRelativeToAbsolute(
+            href.replace(/<|>|"/g, '').trim(),
+            manifestUri,
+        );
 
         const asset = { value };
         for (const key of rest) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@podium/client",
-  "version": "5.2.3",
+  "version": "5.2.5",
   "type": "module",
   "license": "MIT",
   "keywords": [

package/types/utils.d.ts

@@ -2,5 +2,5 @@ export function isHeaderDefined(headers: object, header: string): boolean;
 export function hasManifestChange(item: object): boolean;
 export function validateIncoming(incoming?: object): boolean;
 export function filterAssets<T extends import("@podium/utils").AssetCss | import("@podium/utils").AssetJs>(scope: "content" | "fallback" | "all", assets: T[]): T[];
-export function parseLinkHeaders(headers: any): any[];
+export function parseLinkHeaders(headers: any, manifestUri: any): any[];
 export function toPreloadAssetObjects(assetObjects: any): any;