@smithy/fetch-http-handler 2.5.0 → 3.0.1

package/README.md

@@ -2,3 +2,10 @@
 
 [![NPM version](https://img.shields.io/npm/v/@smithy/fetch-http-handler/latest.svg)](https://www.npmjs.com/package/@smithy/fetch-http-handler)
 [![NPM downloads](https://img.shields.io/npm/dm/@smithy/fetch-http-handler.svg)](https://www.npmjs.com/package/@smithy/fetch-http-handler)
+
+This is the default `requestHandler` used for browser applications.
+Since Node.js introduced experimental Web Streams API in v16.5.0 and made it stable in v21.0.0,
+you can consider using `fetch-http-handler` in Node.js, although it's not recommended.
+
+For the Node.js default `requestHandler` implementation, see instead
+[`@smithy/node-http-handler`](https://www.npmjs.com/package/@smithy/node-http-handler).

package/dist-cjs/index.js

@@ -97,12 +97,19 @@ var _FetchHttpHandler = class _FetchHttpHandler {
     const { port, method } = request;
     const url = `${request.protocol}//${auth}${request.hostname}${port ? `:${port}` : ""}${path}`;
     const body = method === "GET" || method === "HEAD" ? void 0 : request.body;
-    const requestOptions = { body, headers: new Headers(request.headers), method };
+    const requestOptions = {
+      body,
+      headers: new Headers(request.headers),
+      method
+    };
+    if (body) {
+      requestOptions.duplex = "half";
+    }
     if (typeof AbortController !== "undefined") {
-      requestOptions["signal"] = abortSignal;
+      requestOptions.signal = abortSignal;
     }
     if (keepAliveSupport.supported) {
-      requestOptions["keepalive"] = keepAlive;
+      requestOptions.keepalive = keepAlive;
     }
     const fetchRequest = new Request(url, requestOptions);
     const raceOfPromises = [
@@ -176,20 +183,25 @@ async function collectBlob(blob) {
 }
 __name(collectBlob, "collectBlob");
 async function collectStream(stream) {
-  let res = new Uint8Array(0);
+  const chunks = [];
   const reader = stream.getReader();
   let isDone = false;
+  let length = 0;
   while (!isDone) {
     const { done, value } = await reader.read();
     if (value) {
-      const prior = res;
-      res = new Uint8Array(prior.length + value.length);
-      res.set(prior);
-      res.set(value, prior.length);
+      chunks.push(value);
+      length += value.length;
     }
     isDone = done;
   }
-  return res;
+  const collected = new Uint8Array(length);
+  let offset = 0;
+  for (const chunk of chunks) {
+    collected.set(chunk, offset);
+    offset += chunk.length;
+  }
+  return collected;
 }
 __name(collectStream, "collectStream");
 function readToBase64(blob) {

package/dist-es/fetch-http-handler.js

@@ -50,12 +50,19 @@ export class FetchHttpHandler {
         const { port, method } = request;
         const url = `${request.protocol}//${auth}${request.hostname}${port ? `:${port}` : ""}${path}`;
         const body = method === "GET" || method === "HEAD" ? undefined : request.body;
-        const requestOptions = { body, headers: new Headers(request.headers), method: method };
+        const requestOptions = {
+            body,
+            headers: new Headers(request.headers),
+            method: method,
+        };
+        if (body) {
+            requestOptions.duplex = "half";
+        }
         if (typeof AbortController !== "undefined") {
-            requestOptions["signal"] = abortSignal;
+            requestOptions.signal = abortSignal;
         }
         if (keepAliveSupport.supported) {
-            requestOptions["keepalive"] = keepAlive;
+            requestOptions.keepalive = keepAlive;
         }
         const fetchRequest = new Request(url, requestOptions);
         const raceOfPromises = [

package/dist-es/stream-collector.js

@@ -11,20 +11,25 @@ async function collectBlob(blob) {
     return new Uint8Array(arrayBuffer);
 }
 async function collectStream(stream) {
-    let res = new Uint8Array(0);
+    const chunks = [];
     const reader = stream.getReader();
     let isDone = false;
+    let length = 0;
     while (!isDone) {
         const { done, value } = await reader.read();
         if (value) {
-            const prior = res;
-            res = new Uint8Array(prior.length + value.length);
-            res.set(prior);
-            res.set(value, prior.length);
+            chunks.push(value);
+            length += value.length;
         }
         isDone = done;
     }
-    return res;
+    const collected = new Uint8Array(length);
+    let offset = 0;
+    for (const chunk of chunks) {
+        collected.set(chunk, offset);
+        offset += chunk.length;
+    }
+    return collected;
 }
 function readToBase64(blob) {
     return new Promise((resolve, reject) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@smithy/fetch-http-handler",
-  "version": "2.5.0",
+  "version": "3.0.1",
   "description": "Provides a way to make requests",
   "scripts": {
     "build": "concurrently 'yarn:build:cjs' 'yarn:build:es' 'yarn:build:types && yarn build:types:downlevel'",
@@ -13,7 +13,7 @@
     "lint": "eslint -c ../../.eslintrc.js \"src/**/*.ts\"",
     "format": "prettier --config ../../prettier.config.js --ignore-path ../.prettierignore --write \"**/*.{ts,md,json}\"",
     "extract:docs": "api-extractor run --local",
-    "test": "yarn g:jest --coverage --forceExit && karma start karma.conf.js"
+    "test": "yarn g:jest --coverage --forceExit && yarn g:karma start karma.conf.js"
   },
   "author": {
     "name": "AWS SDK for JavaScript Team",
@@ -24,35 +24,18 @@
   "module": "./dist-es/index.js",
   "types": "./dist-types/index.d.ts",
   "dependencies": {
-    "@smithy/protocol-http": "^3.3.0",
-    "@smithy/querystring-builder": "^2.2.0",
-    "@smithy/types": "^2.12.0",
-    "@smithy/util-base64": "^2.3.0",
+    "@smithy/protocol-http": "^4.0.0",
+    "@smithy/querystring-builder": "^3.0.0",
+    "@smithy/types": "^3.0.0",
+    "@smithy/util-base64": "^3.0.0",
     "tslib": "^2.6.2"
   },
   "devDependencies": {
-    "@smithy/abort-controller": "^2.2.0",
-    "@tsconfig/recommended": "1.0.1",
-    "@types/chai-as-promised": "^7.1.2",
-    "chai": "^4.2.0",
-    "chai-as-promised": "^7.1.1",
+    "@smithy/abort-controller": "^3.0.0",
     "concurrently": "7.0.0",
     "downlevel-dts": "0.10.1",
-    "karma": "6.4.0",
-    "karma-chai": "0.1.0",
-    "karma-chrome-launcher": "3.1.1",
-    "karma-coverage": "2.2.1",
-    "karma-env-preprocessor": "0.1.1",
-    "karma-firefox-launcher": "2.1.3",
-    "karma-jasmine": "5.1.0",
-    "karma-mocha": "2.0.1",
-    "karma-sourcemap-loader": "0.3.8",
-    "karma-typescript": "5.5.3",
-    "karma-webpack": "5.0.0",
     "rimraf": "3.0.2",
-    "typedoc": "0.23.23",
-    "webpack": "5.76.0",
-    "webpack-cli": "4.10.0"
+    "typedoc": "0.23.23"
   },
   "typesVersions": {
     "<4.0": {