@digitraffic/common 2024.3.13-1 → 2024.3.14-1

package/dist/__test__/imports.test.mjs

@@ -75,10 +75,6 @@ test('httpserver import ok?', () => {
     const httpserver = import("../test/httpserver.mjs");
     return expect(httpserver).resolves.toBeDefined();
 });
-test('secretsManager import ok?', () => {
-    const secretsManager = import("../test/secrets-manager.mjs");
-    return expect(secretsManager).resolves.toBeDefined();
-});
 test('asserter import ok?', () => {
     const asserter = import("../test/asserter.mjs");
     return expect(asserter).resolves.toBeDefined();
@@ -311,10 +307,12 @@ test('messaging import ok?', () => {
     const messaging = import("../aws/runtime/messaging.mjs");
     return expect(messaging).resolves.toBeDefined();
 });
+/*
+temporary disable, enable after sdk v2 is kicked out
 test('apikey import ok?', () => {
-    const apikey = import("../aws/runtime/apikey.mjs");
-    return expect(apikey).resolves.toBeDefined();
-});
+  const apikey = import("../aws/runtime/apikey.mjs");
+  return expect(apikey).resolves.toBeDefined();
+});*/
 test('environment import ok?', () => {
     const environment = import("../aws/runtime/environment.mjs");
     return expect(environment).resolves.toBeDefined();

package/dist/__test__/secrets/secret-holder.test.mjs

@@ -1,19 +1,29 @@
-import { mockSecret, stubSecretsManager } from "../../test/secrets-manager.mjs";
+import { SecretsManager } from "@aws-sdk/client-secrets-manager";
+import { jest } from "@jest/globals";
 const SECRET_WITH_PREFIX = {
     "prefix.value": "value",
     "prefix.name": "name",
     "wrong.value": "value",
     username: "DB_USER",
 };
-const SECRET_EMPTY = {};
-const stubSM = stubSecretsManager();
-const secretHolder = await import("../../aws/runtime/secrets/secret-holder.mjs");
-const database = await import("../../database/database.mjs");
-const { SecretHolder } = secretHolder;
-const { DatabaseEnvironmentKeys } = database;
+const emptySecret = { $metadata: {} };
+const getSecretValueMock = jest.fn();
+// eslint-disable-next-line @typescript-eslint/no-misused-promises
+jest.spyOn(SecretsManager.prototype, "getSecretValue").mockImplementation(getSecretValueMock);
+const { SecretHolder } = await import("../../aws/runtime/secrets/secret-holder.mjs");
+const { DatabaseEnvironmentKeys } = await import("../../database/database.mjs");
+function mockSecret(secret) {
+    if (!secret) {
+        getSecretValueMock.mockImplementation(() => Promise.resolve(emptySecret));
+    }
+    else {
+        getSecretValueMock.mockImplementation(() => Promise.resolve({ ...emptySecret, ...{ SecretString: JSON.stringify(secret) } }));
+    }
+}
 describe("SecretHolder - tests", () => {
     beforeEach(() => {
         process.env["SECRET_ID"] = "test-id";
+        process.env["AWS_REGION"] = "eu-west-1";
     });
     afterEach(() => {
         // eslint-disable-next-line @typescript-eslint/no-dynamic-delete
@@ -26,10 +36,10 @@ describe("SecretHolder - tests", () => {
         return expect(secret).rejects.toThrow("No secret found!");
     }, 10000);
     test("get - empty secret", () => {
-        mockSecret(SECRET_EMPTY);
+        mockSecret({});
         const holder = SecretHolder.create();
         const secret = holder.get();
-        return expect(secret).resolves.toEqual(SECRET_EMPTY);
+        return expect(secret).resolves.toEqual({});
     });
     test("get - no prefix", () => {
         mockSecret(SECRET_WITH_PREFIX);
@@ -60,27 +70,27 @@ describe("SecretHolder - tests", () => {
     test("get - ttl - do not fetch", async () => {
         mockSecret(SECRET_WITH_PREFIX);
         const holder = SecretHolder.create();
-        const callCount = stubSM.getSecretValue.callCount;
+        const callCount = getSecretValueMock.mock.calls.length;
         await holder.get();
-        expect(stubSM.getSecretValue.callCount).toEqual(callCount + 1);
+        expect(getSecretValueMock).toHaveBeenCalledTimes(callCount + 1);
         // gets cached secret
         await holder.get();
-        expect(stubSM.getSecretValue.callCount).toEqual(callCount + 1);
+        expect(getSecretValueMock).toHaveBeenCalledTimes(callCount + 1);
     });
     test("get - ttl - fetch", async () => {
         mockSecret(SECRET_WITH_PREFIX);
         const holder = new SecretHolder("", "", [], {
             ttl: 1,
         });
-        const callCount = stubSM.getSecretValue.callCount;
+        const callCount = getSecretValueMock.mock.calls.length;
         await holder.get();
-        expect(stubSM.getSecretValue.callCount).toEqual(callCount + 1);
+        expect(getSecretValueMock).toHaveBeenCalledTimes(callCount + 1);
         // cache expires, fetches secret again
         const start = Date.now();
         while (Date.now() < start + 2000)
             ;
         await holder.get();
-        expect(stubSM.getSecretValue.callCount).toEqual(callCount + 2);
+        expect(getSecretValueMock).toHaveBeenCalledTimes(callCount + 2);
     });
 });
 //# sourceMappingURL=secret-holder.test.mjs.map

package/dist/__test__/secrets/secret.test.mjs

@@ -1,12 +1,24 @@
-import { mockSecret, stubSecretsManager } from "../../test/secrets-manager.mjs";
+import { SecretsManager } from "@aws-sdk/client-secrets-manager";
+import { jest } from "@jest/globals";
 const SECRET_ID = "test_secret";
 const SECRET_WITH_PREFIX = {
     "prefix.value": "value",
     "prefix.name": "name",
     "wrong.value": "value",
 };
-const SECRET_EMPTY = {};
-stubSecretsManager();
+const emptySecret = { $metadata: {} };
+const getSecretValueMock = jest.fn();
+// eslint-disable-next-line @typescript-eslint/no-misused-promises
+jest.spyOn(SecretsManager.prototype, "getSecretValue").mockImplementation(getSecretValueMock);
+function mockSecret(secret) {
+    if (!secret) {
+        getSecretValueMock.mockImplementation(() => Promise.resolve(emptySecret));
+    }
+    else {
+        getSecretValueMock.mockImplementation(() => Promise.resolve({ ...emptySecret, ...{ SecretString: JSON.stringify(secret) } }));
+    }
+}
+process.env["AWS_REGION"] = "eu-west-1";
 const secret = await import("../../aws/runtime/secrets/secret.mjs");
 const { getSecret } = secret;
 describe("secret - test", () => {
@@ -17,9 +29,9 @@ describe("secret - test", () => {
         }).rejects.toThrow("No secret found!");
     });
     test("getSecret - empty secret", async () => {
-        mockSecret(SECRET_EMPTY);
+        mockSecret({});
         const secret = await getSecret(SECRET_ID, "");
-        expect(secret).toEqual(SECRET_EMPTY);
+        expect(secret).toEqual({});
     });
     test("getSecret - no prefix", async () => {
         mockSecret(SECRET_WITH_PREFIX);

package/dist/aws/infra/canaries/url-checker.mjs

@@ -38,10 +38,9 @@ export class UrlChecker {
             .withIncludeResponseHeaders(false)
             .withFailedCanaryMetric(true);
     }
-    static create(hostname, apiKeyId) {
-        return getApiKeyFromAPIGateway(apiKeyId).then((apiKey) => {
-            return new UrlChecker(hostname, apiKey.value);
-        });
+    static async create(hostname, apiKeyId) {
+        const apiKey = await getApiKeyFromAPIGateway(apiKeyId);
+        return new UrlChecker(hostname, apiKey.value);
     }
     static createV2() {
         return this.create(getEnvVariable(ENV_HOSTNAME), getEnvVariable(ENV_API_KEY));

package/dist/aws/runtime/apikey.d.mts

@@ -1,2 +1,2 @@
-import type { UpdateApiKeyCommandOutput } from "@aws-sdk/client-api-gateway";
-export declare function getApiKeyFromAPIGateway(keyId: string): Promise<UpdateApiKeyCommandOutput>;
+import type { ApiKey } from "aws-sdk/clients/apigateway.js";
+export declare function getApiKeyFromAPIGateway(keyId: string): Promise<ApiKey>;

package/dist/aws/runtime/apikey.mjs

@@ -1,9 +1,19 @@
-import { APIGateway } from "@aws-sdk/client-api-gateway";
-export function getApiKeyFromAPIGateway(keyId) {
-    const agw = new APIGateway();
-    return agw.getApiKey({
+//import { APIGatewayClient, GetApiKeyCommand } from "@aws-sdk/client-api-gateway";
+//import type { GetApiKeyCommandOutput } from "@aws-sdk/client-api-gateway";
+import { APIGateway } from "aws-sdk";
+export async function getApiKeyFromAPIGateway(keyId) {
+    const ag = new APIGateway();
+    return ag.getApiKey({
+        apiKey: keyId,
+        includeValue: true
+    }).promise();
+    /*
+    const client = new APIGatewayClient();
+    const command = new GetApiKeyCommand({
         apiKey: keyId,
         includeValue: true,
     });
+
+    return client.send(command);*/
 }
 //# sourceMappingURL=apikey.mjs.map

package/dist/aws/runtime/secrets/secret.mjs

@@ -1,6 +1,6 @@
+import { SecretsManager } from "@aws-sdk/client-secrets-manager";
 import { getEnvVariable, getEnvVariableOrElse } from "../../../utils/utils.mjs";
 import { EnvKeys } from "../environment.mjs";
-const { SecretsManager } = await import("@aws-sdk/client-secrets-manager");
 // SECRET_OVERRIDE_AWS_REGION might not have been set before import of
 // secret, so we need to lazy initialize SecretsManager
 let smClient;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@digitraffic/common",
-  "version": "2024.3.13-1",
+  "version": "2024.3.14-1",
   "description": "",
   "type": "module",
   "repository": {
@@ -36,7 +36,6 @@
     "./dist/test/testutils": "./dist/test/testutils.mjs",
     "./dist/test/db-testutils": "./dist/test/db-testutils.mjs",
     "./dist/test/httpserver": "./dist/test/httpserver.mjs",
-    "./dist/test/secrets-manager": "./dist/test/secrets-manager.mjs",
     "./dist/test/asserter": "./dist/test/asserter.mjs",
     "./dist/test/mock-ky": "./dist/test/mock-ky.mjs",
     "./dist/marine/rtz": "./dist/marine/rtz.mjs",
@@ -104,12 +103,12 @@
     "./dist/aws/runtime/digitraffic-integration-response": "./dist/aws/runtime/digitraffic-integration-response.mjs"
   },
   "peerDependencies": {
-    "@aws-sdk/client-s3": "^3.496.0",
-    "@aws-sdk/lib-storage": "^3.496.0",
-    "@aws-sdk/client-secrets-manager": "^3.496.0",
-    "@aws-sdk/client-api-gateway": "^3.496.0",
-    "@aws-sdk/client-sns": "^3.496.0",
-    "@types/geojson": "^7946.0.12",
+    "@aws-sdk/client-s3": "^3.533.0",
+    "@aws-sdk/lib-storage": "^3.533.0",
+    "@aws-sdk/client-secrets-manager": "^3.533.0",
+    "@aws-sdk/client-api-gateway": "^3.533.0",
+    "@aws-sdk/client-sns": "^3.533.0",
+    "@types/geojson": "^7946.0.14",
     "aws-cdk-lib": "^2.132.1",
     "change-case": "5.0.0",
     "constructs": "^10.3.0",
@@ -117,51 +116,52 @@
     "date-fns-tz": "~2.0.0",
     "etag": "^1.8.1",
     "geojson-validation": "^1.0.2",
-    "ky": "^1.2.0",
+    "ky": "^1.2.2",
     "lodash": "~4.17.21",
     "node-ttl": "^0.2.0",
     "pg-native": "^3.0.1",
     "pg-promise": "^11.5.4"
   },
   "devDependencies": {
-    "@aws-sdk/client-s3": "3.496.0",
-    "@aws-sdk/lib-storage": "3.496.0",
-    "@aws-sdk/client-secrets-manager": "3.496.0",
-    "@aws-sdk/client-api-gateway": "3.496.0",
-    "@aws-sdk/client-sns": "3.496.0",
+    "aws-sdk": "2.1577.0",
+    "@aws-sdk/client-s3": "3.533.0",
+    "@aws-sdk/lib-storage": "3.533.0",
+    "@aws-sdk/client-secrets-manager": "3.533.0",
+    "@aws-sdk/client-api-gateway": "3.533.0",
+    "@aws-sdk/client-sns": "3.533.0",
     "@jest/globals": "^29.7.0",
-    "@rushstack/eslint-config": "^3.6.2",
-    "@rushstack/heft": "^0.64.3",
-    "@rushstack/heft-jest-plugin": "^0.11.3",
-    "@rushstack/heft-lint-plugin": "^0.3.3",
-    "@rushstack/heft-typescript-plugin": "^0.3.3",
-    "@types/aws-lambda": "8.10.131",
+    "@rushstack/eslint-config": "^3.6.4",
+    "@rushstack/heft": "^0.66.0",
+    "@rushstack/heft-jest-plugin": "^0.11.21",
+    "@rushstack/heft-lint-plugin": "^0.3.20",
+    "@rushstack/heft-typescript-plugin": "^0.3.20",
+    "@types/aws-lambda": "8.10.136",
     "@types/etag": "1.8.3",
-    "@types/geojson": "7946.0.13",
-    "@types/geojson-validation": "^1.0.2",
-    "@types/jest": "29.5.11",
+    "@types/geojson": "7946.0.14",
+    "@types/geojson-validation": "^1.0.3",
+    "@types/jest": "29.5.12",
     "@types/lodash": "4.14.202",
-    "@types/node": "20.10.7",
-    "@types/sinon": "17.0.2",
+    "@types/node": "20.11.27",
+    "@types/sinon": "17.0.3",
     "@typescript-eslint/eslint-plugin": "~6.18.1",
     "@typescript-eslint/parser": "^6.20.0",
     "aws-cdk-lib": "~2.132.1",
     "change-case": "5.3.0",
     "constructs": "10.3.0",
     "date-fns": "~2.30.0",
-    "date-fns-tz": "~2.0.0",
-    "eslint": "~8.56.0",
+    "date-fns-tz": "~2.0.1",
+    "eslint": "~8.57.0",
     "eslint-config-prettier": "^9.1.0",
     "eslint-plugin-deprecation": "~2.0.0",
     "etag": "^1.8.1",
     "geojson-validation": "^1.0.2",
     "jest": "^29.7.0",
     "jest-junit": "^16.0.0",
-    "ky": "^1.2.0",
+    "ky": "^1.2.2",
     "lodash": "~4.17.21",
     "node-ttl": "^0.2.0",
     "pg-promise": "^11.5.4",
-    "prettier": "^3.2.4",
+    "prettier": "^3.2.5",
     "rimraf": "^5.0.5",
     "sinon": "17.0.1",
     "ts-jest": "^29.1.2",

package/dist/test/secrets-manager.d.mts

@@ -1,10 +0,0 @@
-import { SecretsManager } from "@aws-sdk/client-secrets-manager";
-import sinon from "sinon";
-/**
- * Stub Secrets Manager for tests.  You must call this
- * before you instantiate Secrets Manager(this might happen when you import the function that uses Secrets Manager).
- *
- * To mock the actual secret, call mockSecret()
- */
-export declare function stubSecretsManager(): sinon.SinonStubbedInstance<SecretsManager>;
-export declare function mockSecret<Secret>(secret: Secret): void;

package/dist/test/secrets-manager.mjs

@@ -1,32 +0,0 @@
-import { SecretsManager } from "@aws-sdk/client-secrets-manager";
-import sinon from "sinon";
-import { EnvKeys } from "../aws/runtime/environment.mjs";
-import { setEnvVariable } from "../utils/utils.mjs";
-import { jest } from '@jest/globals';
-setEnvVariable(EnvKeys.AWS_REGION, "eu-west-1");
-const emptySecret = { $metadata: {} };
-const SecretsManagerStubInstance = sinon.createStubInstance(SecretsManager);
-SecretsManagerStubInstance.getSecretValue.resolves(emptySecret);
-/**
- * Stub Secrets Manager for tests.  You must call this
- * before you instantiate Secrets Manager(this might happen when you import the function that uses Secrets Manager).
- *
- * To mock the actual secret, call mockSecret()
- */
-export function stubSecretsManager() {
-    jest.unstable_mockModule("@aws-sdk/client-secrets-manager", function () {
-        return {
-            SecretsManager: sinon.stub().returns(SecretsManagerStubInstance)
-        };
-    });
-    return SecretsManagerStubInstance;
-}
-export function mockSecret(secret) {
-    if (!secret) {
-        SecretsManagerStubInstance.getSecretValue.resolves({ ...emptySecret });
-    }
-    else {
-        SecretsManagerStubInstance.getSecretValue.resolves({ SecretString: JSON.stringify(secret) });
-    }
-}
-//# sourceMappingURL=secrets-manager.mjs.map