@podium/client 5.0.1 → 5.0.3

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+## [5.0.3](https://github.com/podium-lib/client/compare/v5.0.2...v5.0.3) (2023-12-06)
+
+
+### Bug Fixes
+
+* export default now that package is esm ([a1739e8](https://github.com/podium-lib/client/commit/a1739e8bcdde25e3cbb396534705e02411beb7cf))
+
+## [5.0.2](https://github.com/podium-lib/client/compare/v5.0.1...v5.0.2) (2023-12-01)
+
+
+### Bug Fixes
+
+* Consume body on non 200 requests ([ebfc8be](https://github.com/podium-lib/client/commit/ebfc8be4cbb2fcb57ac49febc93fd0381fa9c09f))
+
 ## [5.0.1](https://github.com/podium-lib/client/compare/v5.0.0...v5.0.1) (2023-11-30)
 
 

package/README.md

@@ -21,8 +21,8 @@ $ npm install @podium/client
 Connect to a Podium component server and stream the HTML content:
 
 ```js
-const { HttpIncoming } = require('@podium/utils');
-const Client = require('@podium/client');
+import { HttpIncoming } from '@podium/utils';
+import Client from '@podium/client';
 const client = new Client();
 
 const component = client.register({
@@ -47,8 +47,8 @@ stream.pipe(process.stdout);
 Connect to a podium component server and fetch the HTML content:
 
 ```js
-const { HttpIncoming } = require('@podium/utils');
-const Client = require('@podium/client');
+import { HttpIncoming } from '@podium/utils';
+import Client from '@podium/client';
 const client = new Client();
 
 const component = client.register({
@@ -74,7 +74,7 @@ component
 Create a new Client instance.
 
 ```js
-const Client = require('@podium/client');
+import Client from '@podium/client';
 const client = new Client(options);
 ```
 
@@ -82,7 +82,7 @@ The client instance is iterable and holds a reference to each registered
 resource.
 
 ```js
-const Client = require('@podium/client');
+import Client from '@podium/client';
 const client = new Client();
 
 client.register({ uri: 'http://foo.site.com/manifest.json', name: 'fooBar' });
@@ -117,7 +117,7 @@ Registers a component.
 Example:
 
 ```js
-const Client = require('@podium/client');
+import Client from '@podium/client';
 const client = new Client();
 
 const component = client.register({
@@ -134,7 +134,7 @@ It is stored with the `name` as its property name.
 Example:
 
 ```js
-const Client = require('@podium/client');
+import Client from '@podium/client';
 const client = new Client();
 
 client.register({ uri: 'http://foo.site.com/manifest.json', name: 'fooBar' });
@@ -159,7 +159,7 @@ Retrieve list of all JavaScript references from all registered and fetched
 components.
 
 ```js
-const Client = require('@podium/client');
+import Client from '@podium/client';
 const client = new Client();
 
 const foo = client.register({
@@ -182,7 +182,7 @@ Retrieve a list of all CSS references from all registered and fetched
 components.
 
 ```js
-const Client = require('@podium/client');
+import Client from '@podium/client';
 const client = new Client();
 
 const foo = client.register({
@@ -214,7 +214,7 @@ If a manifest is successfully fetched, this method will resolve with a `true`
 value. If a manifest is not successfully fetched, it will resolve with `false`.
 
 ```js
-const Client = require('@podium/client');
+import Client from '@podium/client';
 const client = new Client();
 
 client.register({ uri: 'http://foo.site.com/manifest.json', name: 'foo' });
@@ -233,7 +233,7 @@ Refreshes the manifests of all registered resources. Does so by calling the
 `.refresh()` method on all resources under the hood.
 
 ```js
-const Client = require('@podium/client');
+import Client from '@podium/client';
 const client = new Client();
 
 client.register({ uri: 'http://foo.site.com/manifest.json', name: 'foo' });
@@ -499,7 +499,7 @@ fallback or the content then it can then be acted upon.
 Example:
 
 ```js
-const Client = require('@podium/client');
+import Client from '@podium/client';
 const client = new Client();
 
 const foo = client.register({

package/client.d.ts

@@ -137,4 +137,4 @@ declare namespace PodiumClient {
     }
 }
 
-export = PodiumClient;
+export default PodiumClient;

package/lib/resolver.content.js

@@ -166,6 +166,8 @@ export default class PodletClientContentResolver {
                     },
                 };
 
+                // Body must be consumed; https://github.com/nodejs/undici/issues/583#issuecomment-855384858
+                await body.text();
                 throw new Boom(errorMessage, errorOptions);
             }
             if (resError) {
@@ -186,7 +188,10 @@ export default class PodletClientContentResolver {
                       js: utils.filterAssets("fallback", outgoing.manifest.js),
                       css: utils.filterAssets("fallback", outgoing.manifest.css),
                   }),
-              );
+                );
+
+                // Body must be consumed; https://github.com/nodejs/undici/issues/583#issuecomment-855384858
+                await body.text();
                 return outgoing;
             }
 
@@ -211,7 +216,7 @@ export default class PodletClientContentResolver {
                 this.#log.info(
                     `podlet version number in header differs from cached version number - aborting request to remote resource for content - resource: ${outgoing.name} - url: ${outgoing.contentUri}`,
                 );
-                // TODO r.abort();
+
                 outgoing.status = 'stale';
                 return outgoing;
             }

package/lib/resolver.fallback.js

@@ -113,6 +113,9 @@ export default class PodletClientFallbackResolver {
                 );
 
                 outgoing.fallback = '';
+
+                // Body must be consumed; https://github.com/nodejs/undici/issues/583#issuecomment-855384858
+                await body.text();
                 return outgoing;
             }
 

package/lib/resolver.manifest.js

@@ -99,6 +99,9 @@ export default class PodletClientManifestResolver {
                 this.#log.warn(
                     `remote resource responded with non 200 http status code for manifest - code: ${statusCode} - resource: ${outgoing.name} - url: ${outgoing.manifestUri}`,
                 );
+
+                // Body must be consumed; https://github.com/nodejs/undici/issues/583#issuecomment-855384858
+                await body.text();
                 return outgoing;
             }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@podium/client",
-  "version": "5.0.1",
+  "version": "5.0.3",
   "type": "module",
   "license": "MIT",
   "keywords": [
@@ -50,7 +50,7 @@
     "@semantic-release/changelog": "6.0.3",
     "@semantic-release/git": "10.0.1",
     "@babel/eslint-parser": "7.23.3",
-    "@semantic-release/github": "9.2.3",
+    "@semantic-release/github": "9.2.4",
     "@semantic-release/npm": "11.0.1",
     "@semantic-release/release-notes-generator": "12.1.0",
     "@sinonjs/fake-timers": "11.2.2",