chore(protocol-http): switch to HttpRequest static clone method (smithy-lang#1345)

* chore(protocol-http): switch to HttpRequest static clone method

* update unit test

Author: kuhe

.changeset/poor-dolls-cheer.md

@@ -0,0 +1,8 @@
+---
+"@smithy/protocol-http": minor
+"@smithy/signature-v4": minor
+"@smithy/core": minor
+"@smithy/experimental-identity-and-auth": patch
+---
+
+switch to static HttpRequest clone method

packages/core/src/util-identity-and-auth/httpAuthSchemes/httpApiKeyAuth.ts

@@ -24,7 +24,7 @@ export class HttpApiKeyAuthSigner implements HttpSigner {
     if (!identity.apiKey) {
       throw new Error("request could not be signed with `apiKey` since the `apiKey` is not defined");
     }
-    const clonedRequest = httpRequest.clone();
+    const clonedRequest = HttpRequest.clone(httpRequest);
     if (signingProperties.in === HttpApiKeyAuthLocation.QUERY) {
       clonedRequest.query[signingProperties.name] = identity.apiKey;
     } else if (signingProperties.in === HttpApiKeyAuthLocation.HEADER) {

packages/core/src/util-identity-and-auth/httpAuthSchemes/httpBearerAuth.ts

@@ -11,7 +11,7 @@ export class HttpBearerAuthSigner implements HttpSigner {
     identity: TokenIdentity,
     signingProperties: Record<string, any>
   ): Promise<IHttpRequest> {
-    const clonedRequest = httpRequest.clone();
+    const clonedRequest = HttpRequest.clone(httpRequest);
     if (!identity.token) {
       throw new Error("request could not be signed with `token` since the `token` is not defined");
     }

packages/experimental-identity-and-auth/src/SigV4Signer.ts

@@ -13,7 +13,7 @@ export class SigV4Signer implements HttpSigner {
     identity: AwsCredentialIdentity,
     signingProperties: Record<string, any>
   ): Promise<IHttpRequest> {
-    const clonedRequest = httpRequest.clone();
+    const clonedRequest = HttpRequest.clone(httpRequest);
     const signer = new SignatureV4({
       applyChecksum: signingProperties.applyChecksum !== undefined ? signingProperties.applyChecksum : true,
       credentials: identity,

packages/experimental-identity-and-auth/src/httpApiKeyAuth.ts

@@ -35,7 +35,7 @@ export class HttpApiKeyAuthSigner implements HttpSigner {
     if (!identity.apiKey) {
       throw new Error("request could not be signed with `apiKey` since the `apiKey` is not defined");
     }
-    const clonedRequest = httpRequest.clone();
+    const clonedRequest = HttpRequest.clone(httpRequest);
     if (signingProperties.in === HttpApiKeyAuthLocation.QUERY) {
       clonedRequest.query[signingProperties.name] = identity.apiKey;
     } else if (signingProperties.in === HttpApiKeyAuthLocation.HEADER) {

packages/experimental-identity-and-auth/src/httpBearerAuth.ts

@@ -14,7 +14,7 @@ export class HttpBearerAuthSigner implements HttpSigner {
     identity: TokenIdentity,
     signingProperties: Record<string, any>
   ): Promise<IHttpRequest> {
-    const clonedRequest = httpRequest.clone();
+    const clonedRequest = HttpRequest.clone(httpRequest);
     if (!identity.token) {
       throw new Error("request could not be signed with `token` since the `token` is not defined");
     }

packages/protocol-http/src/httpRequest.spec.ts

@@ -1,3 +1,5 @@
+import { QueryParameterBag } from "@smithy/types";
+
 import { HttpRequest, IHttpRequest } from "./httpRequest";
 
 describe("HttpRequest", () => {
@@ -44,8 +46,8 @@ describe("HttpRequest", () => {
   it("should maintain a deprecated instance clone method", () => {
     const httpRequestInstance = new HttpRequest(httpRequest);
 
-    const clone1 = httpRequestInstance.clone();
-    const clone2 = httpRequestInstance.clone();
+    const clone1 = HttpRequest.clone(httpRequestInstance);
+    const clone2 = HttpRequest.clone(httpRequestInstance);
 
     expect(httpRequestInstance).toEqual(clone1);
     expect(clone1).toEqual(clone2);
@@ -55,3 +57,60 @@ describe("HttpRequest", () => {
     expect(clone1.body).toBe(clone2.body);
   });
 });
+
+const cloneRequest = HttpRequest.clone;
+
+describe("cloneRequest", () => {
+  const request: IHttpRequest = Object.freeze({
+    method: "GET",
+    protocol: "https:",
+    hostname: "foo.us-west-2.amazonaws.com",
+    path: "/",
+    headers: Object.freeze({
+      foo: "bar",
+      compound: "value 1, value 2",
+    }),
+    query: Object.freeze({
+      fizz: "buzz",
+      snap: ["crackle", "pop"],
+    }),
+  });
+
+  it("should return an object matching the provided request", () => {
+    expect(cloneRequest(request)).toEqual(request);
+  });
+
+  it("should return an object that with a different identity", () => {
+    expect(cloneRequest(request)).not.toBe(request);
+  });
+
+  it("should should deep-copy the headers", () => {
+    const clone = cloneRequest(request);
+
+    delete clone.headers.compound;
+    expect(Object.keys(request.headers)).toEqual(["foo", "compound"]);
+    expect(Object.keys(clone.headers)).toEqual(["foo"]);
+  });
+
+  it("should should deep-copy the query", () => {
+    const clone = cloneRequest(request);
+
+    const { snap } = clone.query as QueryParameterBag;
+    (snap as Array<string>).shift();
+
+    expect((request.query as QueryParameterBag).snap).toEqual(["crackle", "pop"]);
+    expect((clone.query as QueryParameterBag).snap).toEqual(["pop"]);
+  });
+
+  it("should not copy the body", () => {
+    const body = new Uint8Array(16);
+    const req = { ...request, body };
+    const clone = cloneRequest(req);
+
+    expect(clone.body).toBe(req.body);
+  });
+
+  it("should handle requests without defined query objects", () => {
+    expect(cloneRequest({ ...request, query: void 0 }).query).toEqual({});
+  });
+});

packages/signature-v4/package.json

@@ -25,6 +25,7 @@
   "license": "Apache-2.0",
   "dependencies": {
     "@smithy/is-array-buffer": "workspace:^",
+    "@smithy/protocol-http": "workspace:^",
     "@smithy/types": "workspace:^",
     "@smithy/util-hex-encoding": "workspace:^",
     "@smithy/util-middleware": "workspace:^",
@@ -34,7 +35,6 @@
   },
   "devDependencies": {
     "@aws-crypto/sha256-js": "5.2.0",
-    "@smithy/protocol-http": "workspace:^",
     "concurrently": "7.0.0",
     "downlevel-dts": "0.10.1",
     "rimraf": "3.0.2",

packages/signature-v4/src/SignatureV4.spec.ts

@@ -410,7 +410,7 @@ describe("SignatureV4", () => {
     });
 
     it("should sign requests without host header", async () => {
-      const request = minimalRequest.clone();
+      const request = HttpRequest.clone(minimalRequest);
       delete request.headers[HOST_HEADER];
       const { headers } = await signer.sign(request, {
         signingDate: new Date("2000-01-01T00:00:00.000Z"),