@rio-cloud/cdk-v2-constructs 6.3.0 → 6.3.2

package/node_modules/@smithy/node-http-handler/dist-es/node-http-handler.js

@@ -16,7 +16,7 @@ export class NodeHttpHandler {
         }
         return new NodeHttpHandler(instanceOrOptions);
     }
-    static checkSocketUsage(agent, socketWarningTimestamp) {
+    static checkSocketUsage(agent, socketWarningTimestamp, logger = console) {
         const { sockets, requests, maxSockets } = agent;
         if (typeof maxSockets !== "number" || maxSockets === Infinity) {
             return socketWarningTimestamp;
@@ -30,7 +30,9 @@ export class NodeHttpHandler {
                 const socketsInUse = sockets[origin]?.length ?? 0;
                 const requestsEnqueued = requests[origin]?.length ?? 0;
                 if (socketsInUse >= maxSockets && requestsEnqueued >= 2 * maxSockets) {
-                    console.warn("@smithy/node-http-handler:WARN", `socket usage at capacity=${socketsInUse} and ${requestsEnqueued} additional requests are enqueued.`, "See https://docs.aws.amazon.com/sdk-for-javascript/v3/developer-guide/node-configuring-maxsockets.html", "or increase socketAcquisitionWarningTimeout=(millis) in the NodeHttpHandler config.");
+                    logger?.warn?.(`@smithy/node-http-handler:WARN - socket usage at capacity=${socketsInUse} and ${requestsEnqueued} additional requests are enqueued.
+See https://docs.aws.amazon.com/sdk-for-javascript/v3/developer-guide/node-configuring-maxsockets.html
+or increase socketAcquisitionWarningTimeout=(millis) in the NodeHttpHandler config.`);
                     return Date.now();
                 }
             }
@@ -72,6 +74,7 @@ export class NodeHttpHandler {
                 }
                 return new hsAgent({ keepAlive, maxSockets, ...httpsAgent });
             })(),
+            logger: console,
         };
     }
     destroy() {
@@ -92,6 +95,7 @@ export class NodeHttpHandler {
             };
             const reject = async (arg) => {
                 await writeRequestBodyPromise;
+                clearTimeout(socketCheckTimeoutId);
                 _reject(arg);
             };
             if (!this.config) {
@@ -106,7 +110,7 @@ export class NodeHttpHandler {
             const isSSL = request.protocol === "https:";
             const agent = isSSL ? this.config.httpsAgent : this.config.httpAgent;
             socketCheckTimeoutId = setTimeout(() => {
-                this.socketWarningTimestamp = NodeHttpHandler.checkSocketUsage(agent, this.socketWarningTimestamp);
+                this.socketWarningTimestamp = NodeHttpHandler.checkSocketUsage(agent, this.socketWarningTimestamp, this.config.logger);
             }, this.config.socketAcquisitionWarningTimeout ??
                 (this.config.requestTimeout ?? 2000) + (this.config.connectionTimeout ?? 1000));
             const queryString = buildQueryString(request.query || {});
@@ -153,12 +157,20 @@ export class NodeHttpHandler {
             setConnectionTimeout(req, reject, this.config.connectionTimeout);
             setSocketTimeout(req, reject, this.config.requestTimeout);
             if (abortSignal) {
-                abortSignal.onabort = () => {
-                    req.abort();
+                const onAbort = () => {
+                    req.destroy();
                     const abortError = new Error("Request aborted");
                     abortError.name = "AbortError";
                     reject(abortError);
                 };
+                if (typeof abortSignal.addEventListener === "function") {
+                    const signal = abortSignal;
+                    signal.addEventListener("abort", onAbort, { once: true });
+                    req.once("close", () => signal.removeEventListener("abort", onAbort));
+                }
+                else {
+                    abortSignal.onabort = onAbort;
+                }
             }
             const httpAgent = nodeHttpsOptions.agent;
             if (typeof httpAgent === "object" && "keepAlive" in httpAgent) {
@@ -167,7 +179,10 @@ export class NodeHttpHandler {
                     keepAliveMsecs: httpAgent.keepAliveMsecs,
                 });
             }
-            writeRequestBodyPromise = writeRequestBody(req, request, this.config.requestTimeout).catch(_reject);
+            writeRequestBodyPromise = writeRequestBody(req, request, this.config.requestTimeout).catch((e) => {
+                clearTimeout(socketCheckTimeoutId);
+                return _reject(e);
+            });
         });
     }
     updateHttpClientConfig(key, value) {

package/node_modules/@smithy/node-http-handler/dist-es/node-http2-handler.js

@@ -113,12 +113,20 @@ export class NodeHttp2Handler {
                 });
             }
             if (abortSignal) {
-                abortSignal.onabort = () => {
+                const onAbort = () => {
                     req.close();
                     const abortError = new Error("Request aborted");
                     abortError.name = "AbortError";
                     rejectWithDestroy(abortError);
                 };
+                if (typeof abortSignal.addEventListener === "function") {
+                    const signal = abortSignal;
+                    signal.addEventListener("abort", onAbort, { once: true });
+                    req.once("close", () => signal.removeEventListener("abort", onAbort));
+                }
+                else {
+                    abortSignal.onabort = onAbort;
+                }
             }
             req.on("frameError", (type, code, id) => {
                 rejectWithDestroy(new Error(`Frame type id ${type} in stream id ${id} has failed with code ${code}.`));

package/node_modules/@smithy/node-http-handler/dist-types/node-http-handler.d.ts

@@ -1,7 +1,7 @@
 /// <reference types="node" />
 /// <reference types="node" />
 import { HttpHandler, HttpRequest, HttpResponse } from "@smithy/protocol-http";
-import type { NodeHttpHandlerOptions } from "@smithy/types";
+import type { Logger, NodeHttpHandlerOptions } from "@smithy/types";
 import { HttpHandlerOptions, Provider } from "@smithy/types";
 import { Agent as hAgent } from "http";
 import { Agent as hsAgent } from "https";
@@ -23,9 +23,11 @@ export declare class NodeHttpHandler implements HttpHandler<NodeHttpHandlerOptio
      * @internal
      *
      * @param agent - http(s) agent in use by the NodeHttpHandler instance.
+     * @param socketWarningTimestamp - last socket usage check timestamp.
+     * @param logger - channel for the warning.
      * @returns timestamp of last emitted warning.
      */
-    static checkSocketUsage(agent: hAgent | hsAgent, socketWarningTimestamp: number): number;
+    static checkSocketUsage(agent: hAgent | hsAgent, socketWarningTimestamp: number, logger?: Logger): number;
     constructor(options?: NodeHttpHandlerOptions | Provider<NodeHttpHandlerOptions | void>);
     private resolveDefaultConfig;
     destroy(): void;

package/node_modules/@smithy/node-http-handler/dist-types/ts3.4/node-http-handler.d.ts

@@ -1,6 +1,6 @@
 /// <reference types="node" />
 import { HttpHandler, HttpRequest, HttpResponse } from "@smithy/protocol-http";
-import { NodeHttpHandlerOptions } from "@smithy/types";
+import { Logger, NodeHttpHandlerOptions } from "@smithy/types";
 import { HttpHandlerOptions, Provider } from "@smithy/types";
 import { Agent as hAgent } from "http";
 import { Agent as hsAgent } from "https";
@@ -22,9 +22,11 @@ export declare class NodeHttpHandler implements HttpHandler<NodeHttpHandlerOptio
      * @internal
      *
      * @param agent - http(s) agent in use by the NodeHttpHandler instance.
+     * @param socketWarningTimestamp - last socket usage check timestamp.
+     * @param logger - channel for the warning.
      * @returns timestamp of last emitted warning.
      */
-    static checkSocketUsage(agent: hAgent | hsAgent, socketWarningTimestamp: number): number;
+    static checkSocketUsage(agent: hAgent | hsAgent, socketWarningTimestamp: number, logger?: Logger): number;
     constructor(options?: NodeHttpHandlerOptions | Provider<NodeHttpHandlerOptions | void>);
     private resolveDefaultConfig;
     destroy(): void;

package/node_modules/@smithy/node-http-handler/package.json

@@ -1,13 +1,13 @@
 {
   "name": "@smithy/node-http-handler",
-  "version": "3.0.0",
+  "version": "3.1.4",
   "description": "Provides a way to make requests",
   "scripts": {
     "build": "concurrently 'yarn:build:cjs' 'yarn:build:es' 'yarn:build:types && yarn build:types:downlevel'",
     "build:cjs": "node ../../scripts/inline node-http-handler",
     "build:es": "yarn g:tsc -p tsconfig.es.json",
     "build:types": "yarn g:tsc -p tsconfig.types.json",
-    "build:types:downlevel": "downlevel-dts dist-types dist-types/ts3.4",
+    "build:types:downlevel": "rimraf dist-types/ts3.4 && downlevel-dts dist-types dist-types/ts3.4",
     "stage-release": "rimraf ./.release && yarn pack && mkdir ./.release && tar zxvf ./package.tgz --directory ./.release && rm ./package.tgz",
     "clean": "rimraf ./dist-* && rimraf *.tsbuildinfo || exit 0",
     "lint": "eslint -c ../../.eslintrc.js \"src/**/*.ts\"",
@@ -25,10 +25,10 @@
   "module": "./dist-es/index.js",
   "types": "./dist-types/index.d.ts",
   "dependencies": {
-    "@smithy/abort-controller": "^3.0.0",
-    "@smithy/protocol-http": "^4.0.0",
-    "@smithy/querystring-builder": "^3.0.0",
-    "@smithy/types": "^3.0.0",
+    "@smithy/abort-controller": "^3.1.1",
+    "@smithy/protocol-http": "^4.1.0",
+    "@smithy/querystring-builder": "^3.0.3",
+    "@smithy/types": "^3.3.0",
     "tslib": "^2.6.2"
   },
   "devDependencies": {

package/node_modules/@smithy/property-provider/dist-cjs/index.js

@@ -31,14 +31,28 @@ module.exports = __toCommonJS(src_exports);
 
 // src/ProviderError.ts
 var _ProviderError = class _ProviderError extends Error {
-  constructor(message, tryNextLink = true) {
+  constructor(message, options = true) {
+    var _a;
+    let logger;
+    let tryNextLink = true;
+    if (typeof options === "boolean") {
+      logger = void 0;
+      tryNextLink = options;
+    } else if (options != null && typeof options === "object") {
+      logger = options.logger;
+      tryNextLink = options.tryNextLink ?? true;
+    }
     super(message);
-    this.tryNextLink = tryNextLink;
     this.name = "ProviderError";
+    this.tryNextLink = tryNextLink;
     Object.setPrototypeOf(this, _ProviderError.prototype);
+    (_a = logger == null ? void 0 : logger.debug) == null ? void 0 : _a.call(logger, `@smithy/property-provider ${tryNextLink ? "->" : "(!)"} ${message}`);
   }
-  static from(error, tryNextLink = true) {
-    return Object.assign(new this(error.message, tryNextLink), error);
+  /**
+   * @deprecated use new operator.
+   */
+  static from(error, options = true) {
+    return Object.assign(new this(error.message, options), error);
   }
 };
 __name(_ProviderError, "ProviderError");
@@ -46,9 +60,11 @@ var ProviderError = _ProviderError;
 
 // src/CredentialsProviderError.ts
 var _CredentialsProviderError = class _CredentialsProviderError extends ProviderError {
-  constructor(message, tryNextLink = true) {
-    super(message, tryNextLink);
-    this.tryNextLink = tryNextLink;
+  /**
+   * @override
+   */
+  constructor(message, options = true) {
+    super(message, options);
     this.name = "CredentialsProviderError";
     Object.setPrototypeOf(this, _CredentialsProviderError.prototype);
   }
@@ -58,9 +74,11 @@ var CredentialsProviderError = _CredentialsProviderError;
 
 // src/TokenProviderError.ts
 var _TokenProviderError = class _TokenProviderError extends ProviderError {
-  constructor(message, tryNextLink = true) {
-    super(message, tryNextLink);
-    this.tryNextLink = tryNextLink;
+  /**
+   * @override
+   */
+  constructor(message, options = true) {
+    super(message, options);
     this.name = "TokenProviderError";
     Object.setPrototypeOf(this, _TokenProviderError.prototype);
   }

package/node_modules/@smithy/property-provider/dist-es/CredentialsProviderError.js

@@ -1,8 +1,7 @@
 import { ProviderError } from "./ProviderError";
 export class CredentialsProviderError extends ProviderError {
-    constructor(message, tryNextLink = true) {
-        super(message, tryNextLink);
-        this.tryNextLink = tryNextLink;
+    constructor(message, options = true) {
+        super(message, options);
         this.name = "CredentialsProviderError";
         Object.setPrototypeOf(this, CredentialsProviderError.prototype);
     }

package/node_modules/@smithy/property-provider/dist-es/ProviderError.js

@@ -1,11 +1,22 @@
 export class ProviderError extends Error {
-    constructor(message, tryNextLink = true) {
+    constructor(message, options = true) {
+        let logger;
+        let tryNextLink = true;
+        if (typeof options === "boolean") {
+            logger = undefined;
+            tryNextLink = options;
+        }
+        else if (options != null && typeof options === "object") {
+            logger = options.logger;
+            tryNextLink = options.tryNextLink ?? true;
+        }
         super(message);
-        this.tryNextLink = tryNextLink;
         this.name = "ProviderError";
+        this.tryNextLink = tryNextLink;
         Object.setPrototypeOf(this, ProviderError.prototype);
+        logger?.debug?.(`@smithy/property-provider ${tryNextLink ? "->" : "(!)"} ${message}`);
     }
-    static from(error, tryNextLink = true) {
-        return Object.assign(new this(error.message, tryNextLink), error);
+    static from(error, options = true) {
+        return Object.assign(new this(error.message, options), error);
     }
 }

package/node_modules/@smithy/property-provider/dist-es/TokenProviderError.js

@@ -1,8 +1,7 @@
 import { ProviderError } from "./ProviderError";
 export class TokenProviderError extends ProviderError {
-    constructor(message, tryNextLink = true) {
-        super(message, tryNextLink);
-        this.tryNextLink = tryNextLink;
+    constructor(message, options = true) {
+        super(message, options);
         this.name = "TokenProviderError";
         Object.setPrototypeOf(this, TokenProviderError.prototype);
     }

package/node_modules/@smithy/property-provider/dist-types/CredentialsProviderError.d.ts

@@ -1,4 +1,4 @@
-import { ProviderError } from "./ProviderError";
+import { ProviderError, ProviderErrorOptionsType } from "./ProviderError";
 /**
  * @public
  *
@@ -11,7 +11,20 @@ import { ProviderError } from "./ProviderError";
  * ensures the chain will stop if an entirely unexpected error is encountered.
  */
 export declare class CredentialsProviderError extends ProviderError {
-    readonly tryNextLink: boolean;
     name: string;
-    constructor(message: string, tryNextLink?: boolean);
+    /**
+     * @override
+     * @deprecated constructor should be given a logger.
+     */
+    constructor(message: string);
+    /**
+     * @override
+     * @deprecated constructor should be given a logger.
+     */
+    constructor(message: string, tryNextLink: boolean | undefined);
+    /**
+     * @override
+     * This signature is preferred for logging capability.
+     */
+    constructor(message: string, options: ProviderErrorOptionsType);
 }

package/node_modules/@smithy/property-provider/dist-types/ProviderError.d.ts

@@ -1,3 +1,11 @@
+import { Logger } from "@smithy/types";
+/**
+ * @public
+ */
+export type ProviderErrorOptionsType = {
+    tryNextLink?: boolean | undefined;
+    logger?: Logger;
+};
 /**
  * @public
  *
@@ -10,8 +18,22 @@
  * ensures the chain will stop if an entirely unexpected error is encountered.
  */
 export declare class ProviderError extends Error {
-    readonly tryNextLink: boolean;
     name: string;
-    constructor(message: string, tryNextLink?: boolean);
-    static from(error: Error, tryNextLink?: boolean): ProviderError;
+    readonly tryNextLink: boolean;
+    /**
+     * @deprecated constructor should be given a logger.
+     */
+    constructor(message: string);
+    /**
+     * @deprecated constructor should be given a logger.
+     */
+    constructor(message: string, tryNextLink: boolean | undefined);
+    /**
+     * This signature is preferred for logging capability.
+     */
+    constructor(message: string, options: ProviderErrorOptionsType);
+    /**
+     * @deprecated use new operator.
+     */
+    static from(error: Error, options?: boolean | ProviderErrorOptionsType): ProviderError;
 }

package/node_modules/@smithy/property-provider/dist-types/TokenProviderError.d.ts

@@ -1,4 +1,4 @@
-import { ProviderError } from "./ProviderError";
+import { ProviderError, ProviderErrorOptionsType } from "./ProviderError";
 /**
  * @public
  *
@@ -11,7 +11,20 @@ import { ProviderError } from "./ProviderError";
  * ensures the chain will stop if an entirely unexpected error is encountered.
  */
 export declare class TokenProviderError extends ProviderError {
-    readonly tryNextLink: boolean;
     name: string;
-    constructor(message: string, tryNextLink?: boolean);
+    /**
+     * @override
+     * @deprecated constructor should be given a logger.
+     */
+    constructor(message: string);
+    /**
+     * @override
+     * @deprecated constructor should be given a logger.
+     */
+    constructor(message: string, tryNextLink: boolean | undefined);
+    /**
+     * @override
+     * This signature is preferred for logging capability.
+     */
+    constructor(message: string, options: ProviderErrorOptionsType);
 }

package/node_modules/@smithy/property-provider/dist-types/ts3.4/CredentialsProviderError.d.ts

@@ -1,4 +1,4 @@
-import { ProviderError } from "./ProviderError";
+import { ProviderError, ProviderErrorOptionsType } from "./ProviderError";
 /**
  * @public
  *
@@ -11,7 +11,20 @@ import { ProviderError } from "./ProviderError";
  * ensures the chain will stop if an entirely unexpected error is encountered.
  */
 export declare class CredentialsProviderError extends ProviderError {
-    readonly tryNextLink: boolean;
     name: string;
-    constructor(message: string, tryNextLink?: boolean);
+    /**
+     * @override
+     * @deprecated constructor should be given a logger.
+     */
+    constructor(message: string);
+    /**
+     * @override
+     * @deprecated constructor should be given a logger.
+     */
+    constructor(message: string, tryNextLink: boolean | undefined);
+    /**
+     * @override
+     * This signature is preferred for logging capability.
+     */
+    constructor(message: string, options: ProviderErrorOptionsType);
 }

package/node_modules/@smithy/property-provider/dist-types/ts3.4/ProviderError.d.ts

@@ -1,3 +1,11 @@
+import { Logger } from "@smithy/types";
+/**
+ * @public
+ */
+export type ProviderErrorOptionsType = {
+    tryNextLink?: boolean | undefined;
+    logger?: Logger;
+};
 /**
  * @public
  *
@@ -10,8 +18,22 @@
  * ensures the chain will stop if an entirely unexpected error is encountered.
  */
 export declare class ProviderError extends Error {
-    readonly tryNextLink: boolean;
     name: string;
-    constructor(message: string, tryNextLink?: boolean);
-    static from(error: Error, tryNextLink?: boolean): ProviderError;
+    readonly tryNextLink: boolean;
+    /**
+     * @deprecated constructor should be given a logger.
+     */
+    constructor(message: string);
+    /**
+     * @deprecated constructor should be given a logger.
+     */
+    constructor(message: string, tryNextLink: boolean | undefined);
+    /**
+     * This signature is preferred for logging capability.
+     */
+    constructor(message: string, options: ProviderErrorOptionsType);
+    /**
+     * @deprecated use new operator.
+     */
+    static from(error: Error, options?: boolean | ProviderErrorOptionsType): ProviderError;
 }

package/node_modules/@smithy/property-provider/dist-types/ts3.4/TokenProviderError.d.ts

@@ -1,4 +1,4 @@
-import { ProviderError } from "./ProviderError";
+import { ProviderError, ProviderErrorOptionsType } from "./ProviderError";
 /**
  * @public
  *
@@ -11,7 +11,20 @@ import { ProviderError } from "./ProviderError";
  * ensures the chain will stop if an entirely unexpected error is encountered.
  */
 export declare class TokenProviderError extends ProviderError {
-    readonly tryNextLink: boolean;
     name: string;
-    constructor(message: string, tryNextLink?: boolean);
+    /**
+     * @override
+     * @deprecated constructor should be given a logger.
+     */
+    constructor(message: string);
+    /**
+     * @override
+     * @deprecated constructor should be given a logger.
+     */
+    constructor(message: string, tryNextLink: boolean | undefined);
+    /**
+     * @override
+     * This signature is preferred for logging capability.
+     */
+    constructor(message: string, options: ProviderErrorOptionsType);
 }

package/node_modules/@smithy/property-provider/package.json

@@ -1,12 +1,12 @@
 {
   "name": "@smithy/property-provider",
-  "version": "3.0.0",
+  "version": "3.1.3",
   "scripts": {
     "build": "concurrently 'yarn:build:cjs' 'yarn:build:es' 'yarn:build:types && yarn build:types:downlevel'",
     "build:cjs": "node ../../scripts/inline property-provider",
     "build:es": "yarn g:tsc -p tsconfig.es.json",
     "build:types": "yarn g:tsc -p tsconfig.types.json",
-    "build:types:downlevel": "downlevel-dts dist-types dist-types/ts3.4",
+    "build:types:downlevel": "rimraf dist-types/ts3.4 && downlevel-dts dist-types dist-types/ts3.4",
     "stage-release": "rimraf ./.release && yarn pack && mkdir ./.release && tar zxvf ./package.tgz --directory ./.release && rm ./package.tgz",
     "clean": "rimraf ./dist-* && rimraf *.tsbuildinfo || exit 0",
     "lint": "eslint -c ../../.eslintrc.js \"src/**/*.ts\"",
@@ -22,7 +22,7 @@
   },
   "license": "Apache-2.0",
   "dependencies": {
-    "@smithy/types": "^3.0.0",
+    "@smithy/types": "^3.3.0",
     "tslib": "^2.6.2"
   },
   "engines": {

package/node_modules/@smithy/protocol-http/dist-cjs/index.js

@@ -24,6 +24,7 @@ __export(src_exports, {
   Fields: () => Fields,
   HttpRequest: () => HttpRequest,
   HttpResponse: () => HttpResponse,
+  IHttpRequest: () => import_types.HttpRequest,
   getHttpHandlerExtensionConfiguration: () => getHttpHandlerExtensionConfiguration,
   isValidHostname: () => isValidHostname,
   resolveHttpHandlerRuntimeConfig: () => resolveHttpHandlerRuntimeConfig
@@ -156,6 +157,7 @@ __name(_Fields, "Fields");
 var Fields = _Fields;
 
 // src/httpRequest.ts
+
 var _HttpRequest = class _HttpRequest {
   constructor(options) {
     this.method = options.method || "GET";
@@ -170,20 +172,40 @@ var _HttpRequest = class _HttpRequest {
     this.password = options.password;
     this.fragment = options.fragment;
   }
+  /**
+   * Note: this does not deep-clone the body.
+   */
+  static clone(request) {
+    const cloned = new _HttpRequest({
+      ...request,
+      headers: { ...request.headers }
+    });
+    if (cloned.query) {
+      cloned.query = cloneQuery(cloned.query);
+    }
+    return cloned;
+  }
+  /**
+   * This method only actually asserts that request is the interface {@link IHttpRequest},
+   * and not necessarily this concrete class. Left in place for API stability.
+   *
+   * Do not call instance methods on the input of this function, and
+   * do not assume it has the HttpRequest prototype.
+   */
   static isInstance(request) {
-    if (!request)
+    if (!request) {
       return false;
+    }
     const req = request;
     return "method" in req && "protocol" in req && "hostname" in req && "path" in req && typeof req["query"] === "object" && typeof req["headers"] === "object";
   }
+  /**
+   * @deprecated use static HttpRequest.clone(request) instead. It's not safe to call
+   * this method because {@link HttpRequest.isInstance} incorrectly
+   * asserts that IHttpRequest (interface) objects are of type HttpRequest (class).
+   */
   clone() {
-    const cloned = new _HttpRequest({
-      ...this,
-      headers: { ...this.headers }
-    });
-    if (cloned.query)
-      cloned.query = cloneQuery(cloned.query);
-    return cloned;
+    return _HttpRequest.clone(this);
   }
 };
 __name(_HttpRequest, "HttpRequest");

package/node_modules/@smithy/protocol-http/dist-es/httpRequest.js

@@ -16,9 +16,20 @@ export class HttpRequest {
         this.password = options.password;
         this.fragment = options.fragment;
     }
+    static clone(request) {
+        const cloned = new HttpRequest({
+            ...request,
+            headers: { ...request.headers },
+        });
+        if (cloned.query) {
+            cloned.query = cloneQuery(cloned.query);
+        }
+        return cloned;
+    }
     static isInstance(request) {
-        if (!request)
+        if (!request) {
             return false;
+        }
         const req = request;
         return ("method" in req &&
             "protocol" in req &&
@@ -28,13 +39,7 @@ export class HttpRequest {
             typeof req["headers"] === "object");
     }
     clone() {
-        const cloned = new HttpRequest({
-            ...this,
-            headers: { ...this.headers },
-        });
-        if (cloned.query)
-            cloned.query = cloneQuery(cloned.query);
-        return cloned;
+        return HttpRequest.clone(this);
     }
 }
 function cloneQuery(query) {

package/node_modules/@smithy/protocol-http/dist-types/httpRequest.d.ts

@@ -2,8 +2,23 @@ import { HeaderBag, HttpMessage, HttpRequest as IHttpRequest, QueryParameterBag,
 type HttpRequestOptions = Partial<HttpMessage> & Partial<URI> & {
     method?: string;
 };
+/**
+ * Use the distinct IHttpRequest interface from @smithy/types instead.
+ * This should not be used due to
+ * overlapping with the concrete class' name.
+ *
+ * This is not marked deprecated since that would mark the concrete class
+ * deprecated as well.
+ */
 export interface HttpRequest extends IHttpRequest {
 }
+/**
+ * @public
+ */
+export { IHttpRequest };
+/**
+ * @public
+ */
 export declare class HttpRequest implements HttpMessage, URI {
     method: string;
     protocol: string;
@@ -17,7 +32,22 @@ export declare class HttpRequest implements HttpMessage, URI {
     fragment?: string;
     body?: any;
     constructor(options: HttpRequestOptions);
+    /**
+     * Note: this does not deep-clone the body.
+     */
+    static clone(request: IHttpRequest): HttpRequest;
+    /**
+     * This method only actually asserts that request is the interface {@link IHttpRequest},
+     * and not necessarily this concrete class. Left in place for API stability.
+     *
+     * Do not call instance methods on the input of this function, and
+     * do not assume it has the HttpRequest prototype.
+     */
     static isInstance(request: unknown): request is HttpRequest;
+    /**
+     * @deprecated use static HttpRequest.clone(request) instead. It's not safe to call
+     * this method because {@link HttpRequest.isInstance} incorrectly
+     * asserts that IHttpRequest (interface) objects are of type HttpRequest (class).
+     */
     clone(): HttpRequest;
 }
-export {};