chore(deps-dev): bump happy-dom from 14.12.3 to 20.0.2 (#1749)

Bumps [happy-dom](https://github.com/capricorn86/happy-dom) from 14.12.3 to 20.0.2.
- [Release notes](https://github.com/capricorn86/happy-dom/releases)
- [Commits](capricorn86/happy-dom@v14.12.3...v20.0.2)

---
updated-dependencies:
- dependency-name: happy-dom
  dependency-version: 20.0.2
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Author: dependabot[bot]

.changeset/cold-dolls-applaud.md

@@ -0,0 +1,5 @@
+---
+"@smithy/config-resolver": minor
+---
+
+validate region is hostname component

package.json

@@ -48,7 +48,7 @@
     "eslint-plugin-simple-import-sort": "7.0.0",
     "eslint-plugin-tsdoc": "0.2.17",
     "get-port": "^7.1.0",
-    "happy-dom": "14.12.3",
+    "happy-dom": "20.0.2",
     "husky": "^4.2.3",
     "jest": "29.7.0",
     "prettier": "3.2.5",

packages/config-resolver/package.json

@@ -28,6 +28,7 @@
     "@smithy/node-config-provider": "workspace:^",
     "@smithy/types": "workspace:^",
     "@smithy/util-config-provider": "workspace:^",
+    "@smithy/util-endpoints": "workspace:^",
     "@smithy/util-middleware": "workspace:^",
     "tslib": "^2.6.2"
   },

packages/config-resolver/src/regionConfig/checkRegion.spec.ts

@@ -0,0 +1,76 @@
+import { describe, test as it, expect, vi } from "vitest";
+import { isValidHostLabel } from "@smithy/util-endpoints";
+import { checkRegion } from "./checkRegion";
+
+describe("checkRegion", () => {
+  const acceptedRegionExamples = [
+    "us-east-1",
+    "ap-east-1",
+    "ap-southeast-4",
+    "ap-northeast-3",
+    "ap-northeast-1",
+    "eu-west-2",
+    "il-central-1",
+    "mx-central-1",
+    "eu-isoe-santaclaus-125",
+    "us-iso-reindeer-3000",
+    "eusc-de-gingerbread-8000",
+    "abcd",
+    "12345",
+  ];
+
+  it("does not throw when the region is a valid host label", () => {
+    for (const region of acceptedRegionExamples) {
+      expect(() => checkRegion(region)).not.toThrow();
+    }
+  });
+
+  it("throws when the region is not a valid host label", () => {
+    for (const region of [
+      "us-east-1-",
+      "a".repeat(64),
+      "-us-east-1",
+      "",
+      "!",
+      "@",
+      "#",
+      "$",
+      "%",
+      "^",
+      "&",
+      "*",
+      "(",
+      ")",
+      ".",
+      "[",
+      "]",
+      ";",
+      `'`,
+      "?",
+      "/",
+      "\\",
+      "|",
+      "+-*/",
+    ]) {
+      expect(() => checkRegion(region)).toThrow(
+        `Region not accepted: region="${region}" is not a valid hostname component.`
+      );
+    }
+  });
+
+  it("caches accepted regions", () => {
+    const di = {
+      isValidHostLabel,
+    };
+    for (const region of acceptedRegionExamples) {
+      expect(() => checkRegion(region, di.isValidHostLabel)).not.toThrow();
+    }
+    vi.spyOn(di, "isValidHostLabel").mockImplementation(isValidHostLabel);
+    for (const region of acceptedRegionExamples) {
+      expect(() => checkRegion(region, di.isValidHostLabel)).not.toThrow();
+    }
+    expect(di.isValidHostLabel).toHaveBeenCalledTimes(0);
+    expect(() => checkRegion("oh-canada", di.isValidHostLabel)).not.toThrow();
+    expect(di.isValidHostLabel).toHaveBeenCalledTimes(1);
+  });
+});

packages/config-resolver/src/regionConfig/checkRegion.ts

@@ -0,0 +1,22 @@
+import { isValidHostLabel } from "@smithy/util-endpoints";
+
+/**
+ * @internal
+ */
+const validRegions = new Set<string>();
+
+/**
+ * Checks whether region can be a host component.
+ *
+ * @param region - to check.
+ * @param check - checking function.
+ *
+ * @internal
+ */
+export const checkRegion = (region: string, check = isValidHostLabel) => {
+  if (!validRegions.has(region) && !check(region)) {
+    throw new Error(`Region not accepted: region="${region}" is not a valid hostname component.`);
+  } else {
+    validRegions.add(region);
+  }
+};

packages/config-resolver/src/regionConfig/resolveRegionConfig.ts

@@ -2,6 +2,7 @@ import type { Provider } from "@smithy/types";
 
 import { getRealRegion } from "./getRealRegion";
 import { isFipsRegion } from "./isFipsRegion";
+import { checkRegion } from "./checkRegion";
 
 /**
  * @public
@@ -47,11 +48,10 @@ export const resolveRegionConfig = <T>(input: T & RegionInputConfig & Previously
 
   return Object.assign(input, {
     region: async () => {
-      if (typeof region === "string") {
-        return getRealRegion(region);
-      }
-      const providedRegion = await region();
-      return getRealRegion(providedRegion);
+      const providedRegion = typeof region === "function" ? await region() : region;
+      const realRegion = getRealRegion(providedRegion);
+      checkRegion(realRegion);
+      return realRegion;
     },
     useFipsEndpoint: async () => {
       const providedRegion = typeof region === "string" ? region : await region();

packages/util-endpoints/src/lib/isValidHostLabel.spec.ts

@@ -32,7 +32,7 @@ describe(isValidHostLabel.name, () => {
       expect(isValidHostLabel(hostLabelToTest, true)).toBe(output);
     });
 
-    describe("returns false is any subdomain is invalid", () => {
+    describe("returns false if any subdomain is invalid", () => {
       const validHostLabel = testCases
         .filter(([outputEntry]) => outputEntry === true)
         .map(([, value]) => value)

yarn.lock

@@ -2514,6 +2514,7 @@ __metadata:
     "@smithy/node-config-provider": "workspace:^"
     "@smithy/types": "workspace:^"
     "@smithy/util-config-provider": "workspace:^"
+    "@smithy/util-endpoints": "workspace:^"
     "@smithy/util-middleware": "workspace:^"
     concurrently: "npm:7.0.0"
     downlevel-dts: "npm:0.10.1"
@@ -3732,6 +3733,15 @@ __metadata:
   languageName: node
   linkType: hard
 
+"@types/node@npm:^20.0.0":
+  version: 20.19.21
+  resolution: "@types/node@npm:20.19.21"
+  dependencies:
+    undici-types: "npm:~6.21.0"
+  checksum: 10c0/a5bedcc71b363abfa425c01e69696820736614dfba03d2363df6d306d2ee6f7619b370c72b4ec75d992e5d76eaccdcdeb362276cac1cd01756db6dec2eb8227c
+  languageName: node
+  linkType: hard
+
 "@types/normalize-package-data@npm:^2.4.0":
   version: 2.4.4
   resolution: "@types/normalize-package-data@npm:2.4.4"
@@ -3774,6 +3784,13 @@ __metadata:
   languageName: node
   linkType: hard
 
+"@types/whatwg-mimetype@npm:^3.0.2":
+  version: 3.0.2
+  resolution: "@types/whatwg-mimetype@npm:3.0.2"
+  checksum: 10c0/dad39d1e4abe760a0a963c84bbdbd26b1df0eb68aff83bdf6ecbb50ad781ead777f6906d19a87007790b750f7500a12e5624d31fc6a1529d14bd19b5c3a316d1
+  languageName: node
+  linkType: hard
+
 "@types/yargs-parser@npm:*":
   version: 21.0.0
   resolution: "@types/yargs-parser@npm:21.0.0"
@@ -6732,14 +6749,14 @@ __metadata:
   languageName: node
   linkType: hard
 
-"happy-dom@npm:14.12.3":
-  version: 14.12.3
-  resolution: "happy-dom@npm:14.12.3"
+"happy-dom@npm:20.0.2":
+  version: 20.0.2
+  resolution: "happy-dom@npm:20.0.2"
   dependencies:
-    entities: "npm:^4.5.0"
-    webidl-conversions: "npm:^7.0.0"
+    "@types/node": "npm:^20.0.0"
+    "@types/whatwg-mimetype": "npm:^3.0.2"
     whatwg-mimetype: "npm:^3.0.0"
-  checksum: 10c0/72e817bc9d03c4c0dbb794f8fb5b69aac4b4dbe0ee789a6b6a02a60298a9aa99bac766967a5c642153e5c95c02489437f878de5641ed64fd13b2271c56805df5
+  checksum: 10c0/d4449e087cb7566fb493f9e6ee99221c64177732bf944ec3e2f464033107832c637e201e3e78e81a8b35871d41aa510806f3f1a2f18b924fc6120226f337a845
   languageName: node
   linkType: hard
 
@@ -10067,7 +10084,7 @@ __metadata:
     eslint-plugin-tsdoc: "npm:0.2.17"
     get-port: "npm:^7.1.0"
     glob: "npm:^7.1.6"
-    happy-dom: "npm:14.12.3"
+    happy-dom: "npm:20.0.2"
     husky: "npm:^4.2.3"
     jest: "npm:29.7.0"
     prettier: "npm:3.2.5"
@@ -10991,6 +11008,13 @@ __metadata:
   languageName: node
   linkType: hard
 
+"undici-types@npm:~6.21.0":
+  version: 6.21.0
+  resolution: "undici-types@npm:6.21.0"
+  checksum: 10c0/c01ed51829b10aa72fc3ce64b747f8e74ae9b60eafa19a7b46ef624403508a54c526ffab06a14a26b3120d055e1104d7abe7c9017e83ced038ea5cf52f8d5e04
+  languageName: node
+  linkType: hard
+
 "unique-filename@npm:^2.0.0":
   version: 2.0.1
   resolution: "unique-filename@npm:2.0.1"
@@ -11339,13 +11363,6 @@ __metadata:
   languageName: node
   linkType: hard
 
-"webidl-conversions@npm:^7.0.0":
-  version: 7.0.0
-  resolution: "webidl-conversions@npm:7.0.0"
-  checksum: 10c0/228d8cb6d270c23b0720cb2d95c579202db3aaf8f633b4e9dd94ec2000a04e7e6e43b76a94509cdb30479bd00ae253ab2371a2da9f81446cc313f89a4213a2c4
-  languageName: node
-  linkType: hard
-
 "webpack-cli@npm:^6.0.1":
   version: 6.0.1
   resolution: "webpack-cli@npm:6.0.1"