@digitraffic/common 2023.10.11-1 → 2023.11.21-1

package/dist/aws/infra/api/integration.d.ts

@@ -6,28 +6,28 @@ interface ApiParameter {
     type: ParameterType;
     name: string;
 }
-export declare class DigitrafficIntegration {
+export declare class DigitrafficIntegration<T extends string> {
     readonly lambda: IFunction;
     readonly mediaType: MediaType;
     readonly parameters: ApiParameter[];
     readonly sunset?: string;
     constructor(lambda: IFunction, mediaType?: MediaType, sunset?: string);
-    addPathParameter(...names: string[]): this;
-    addQueryParameter(...names: string[]): this;
-    addMultiValueQueryParameter(...names: string[]): this;
+    addPathParameter(...names: T[]): this;
+    addQueryParameter(...names: T[]): this;
+    addMultiValueQueryParameter(...names: T[]): this;
     /**
      * Note that context parameter values needs to be in json format as they will be parsed in template as json.
      * See createRequestTemplates below.
      * @param names for the parameters
      * @returns
      */
-    addContextParameter(...names: string[]): this;
+    addContextParameter(...names: T[]): this;
     /**
      * Do not use Authorization header as that will be consumed by ApiGW.
      * If Authorization header is needed, use lambda authorizers.
      * @param names for the headers
      */
-    addHeaderParameter(...names: string[]): this;
+    addHeaderParameter(...names: T[]): this;
     build(): LambdaIntegration;
     createRequestParameters(): Record<string, string>;
     createRequestTemplates(): Record<string, string>;

package/dist/aws/infra/canaries/canary-alarm.d.ts

@@ -1,6 +1,6 @@
 import { Construct } from "constructs";
 import { CanaryParameters } from "./canary-parameters";
-import { Canary } from "@aws-cdk/aws-synthetics-alpha";
+import { Canary } from "aws-cdk-lib/aws-synthetics";
 export declare class CanaryAlarm {
     constructor(stack: Construct, canary: Canary, params: CanaryParameters);
 }

package/dist/aws/infra/canaries/canary-parameters.d.ts

@@ -1,4 +1,4 @@
-import { Schedule } from "@aws-cdk/aws-synthetics-alpha";
+import { Schedule } from "aws-cdk-lib/aws-synthetics";
 /** Optional env parameters for canary */
 type CanaryEnv = Record<string, string>;
 export interface CanaryParameters {

package/dist/aws/infra/canaries/canary.d.ts

@@ -1,4 +1,4 @@
-import { Canary } from "@aws-cdk/aws-synthetics-alpha";
+import { Canary } from "aws-cdk-lib/aws-synthetics";
 import { Role } from "aws-cdk-lib/aws-iam";
 import { CanaryParameters } from "./canary-parameters";
 import { Construct } from "constructs";

package/dist/aws/infra/canaries/canary.js

@@ -2,15 +2,15 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.DigitrafficCanary = void 0;
 const aws_cdk_lib_1 = require("aws-cdk-lib");
-const aws_synthetics_alpha_1 = require("@aws-cdk/aws-synthetics-alpha");
+const aws_synthetics_1 = require("aws-cdk-lib/aws-synthetics");
 const canary_alarm_1 = require("./canary-alarm");
-class DigitrafficCanary extends aws_synthetics_alpha_1.Canary {
+class DigitrafficCanary extends aws_synthetics_1.Canary {
     constructor(scope, canaryName, role, params, environmentVariables) {
         super(scope, canaryName, {
-            runtime: aws_synthetics_alpha_1.Runtime.SYNTHETICS_NODEJS_PUPPETEER_4_0,
+            runtime: aws_synthetics_1.Runtime.SYNTHETICS_NODEJS_PUPPETEER_4_0,
             role,
-            test: aws_synthetics_alpha_1.Test.custom({
-                code: new aws_synthetics_alpha_1.AssetCode("dist", {
+            test: aws_synthetics_1.Test.custom({
+                code: new aws_synthetics_1.AssetCode("dist", {
                     exclude: ["lambda", "out", "canaries"],
                 }),
                 handler: params.handler,
@@ -20,7 +20,7 @@ class DigitrafficCanary extends aws_synthetics_alpha_1.Canary {
                 ...params.canaryEnv,
             },
             canaryName,
-            schedule: params.schedule ?? aws_synthetics_alpha_1.Schedule.rate(aws_cdk_lib_1.Duration.minutes(15)),
+            schedule: params.schedule ?? aws_synthetics_1.Schedule.rate(aws_cdk_lib_1.Duration.minutes(15)),
         });
         this.artifactsBucket.grantWrite(role);
         new canary_alarm_1.CanaryAlarm(scope, this, params);

package/dist/aws/infra/canaries/database-checker.js

@@ -5,6 +5,7 @@ const database_1 = require("../../../database/database");
 const proxy_holder_1 = require("../../runtime/secrets/proxy-holder");
 const rds_holder_1 = require("../../runtime/secrets/rds-holder");
 const utils_1 = require("../../../utils/utils");
+const dt_logger_default_1 = require("../../runtime/dt-logger-default");
 // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-var-requires
 const synthetics = require("Synthetics");
 class DatabaseCheck {
@@ -97,7 +98,10 @@ class DatabaseCountChecker {
         await this.credentialsFunction();
         await (0, database_1.inDatabaseReadonly)(async (db) => {
             for (const check of this.checks) {
-                console.info("canary checking sql " + check.sql);
+                dt_logger_default_1.logger.info({
+                    method: "DatabaseCountChecker.expect",
+                    message: "Running sql: " + check.sql,
+                });
                 const value = await db.one(check.sql);
                 const checkFunction = () => {
                     check.check(value);

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

@@ -10,6 +10,8 @@ const apikey_1 = require("../../runtime/apikey");
 const geometry_1 = require("../../../utils/geometry");
 const utils_1 = require("../../../utils/utils");
 const canary_keys_1 = require("./canary-keys");
+const dt_logger_default_1 = require("../../runtime/dt-logger-default");
+const logging_1 = require("../../../utils/logging");
 exports.API_KEY_HEADER = "x-api-key";
 const baseHeaders = {
     "Digitraffic-User": "internal-digitraffic-canary",
@@ -88,7 +90,10 @@ class UrlChecker {
         // eslint-disable-next-line @typescript-eslint/prefer-optional-chain
         !this.requestOptions.headers ||
             !this.requestOptions.headers[exports.API_KEY_HEADER]) {
-            console.error("No api key defined");
+            dt_logger_default_1.logger.error({
+                method: "UrlChecker.expect403WithoutApiKey",
+                message: "No Api-key defined",
+            });
         }
         const requestOptions = {
             ...this.requestOptions,
@@ -112,7 +117,7 @@ async function getResponseBody(response) {
             return zlib.gunzipSync(body).toString();
         }
         catch (e) {
-            console.info("error " + JSON.stringify(e));
+            (0, logging_1.logException)(dt_logger_default_1.logger, e);
         }
     }
     return body.toString();

package/dist/aws/infra/stack/monitoredfunction.js

@@ -1,12 +1,15 @@
 "use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.MonitoredDBFunction = exports.MonitoredFunction = void 0;
 const aws_lambda_1 = require("aws-cdk-lib/aws-lambda");
 const aws_cloudwatch_actions_1 = require("aws-cdk-lib/aws-cloudwatch-actions");
 const aws_cloudwatch_1 = require("aws-cdk-lib/aws-cloudwatch");
 const lambda_configs_1 = require("../stack/lambda-configs");
-const change_case_1 = require("change-case");
 const subscription_1 = require("../stack/subscription");
+const lodash_1 = __importDefault(require("lodash"));
 /**
  * Creates a Lambda function that monitors default CloudWatch Lambda metrics with CloudWatch Alarms.
  */
@@ -40,7 +43,11 @@ class MonitoredFunction extends aws_lambda_1.Function {
      */
     static createV2(stack, name, environment, functionParameters) {
         const functionName = functionParameters?.functionName ??
-            `${stack.configuration.shortName}-${(0, change_case_1.pascalCase)(name)}`;
+            `${stack.configuration.shortName}-${lodash_1.default.chain(name)
+                .camelCase()
+                .startCase()
+                .replace(/\s/g, "")
+                .value()}`;
         const functionProps = (0, lambda_configs_1.databaseFunctionProps)(stack, environment, functionName, name, functionParameters);
         return MonitoredFunction.create(stack, functionName, functionProps, functionParameters);
     }
@@ -130,7 +137,11 @@ class MonitoredDBFunction {
      */
     static create(stack, name, environment, functionParameters) {
         const functionName = functionParameters?.functionName ??
-            `${stack.configuration.shortName}-${(0, change_case_1.pascalCase)(name)}`;
+            `${stack.configuration.shortName}-${lodash_1.default.chain(name)
+                .camelCase()
+                .startCase()
+                .replace(/\s/g, "")
+                .value()}`;
         const env = environment ? environment : stack.createLambdaEnvironment();
         const functionProps = (0, lambda_configs_1.databaseFunctionProps)(stack, env, functionName, name, functionParameters);
         const mf = MonitoredFunction.create(stack, functionName, functionProps, functionParameters);

package/dist/aws/infra/stack/rest_apis.js

@@ -1,4 +1,7 @@
 "use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.createIpRestrictionPolicyDocument = exports.createDefaultPolicyDocument = exports.createRestApi = exports.setReturnCodeForMissingAuthenticationToken = exports.add401Support = exports.add404Support = exports.DigitrafficRestApi = void 0;
 const aws_apigateway_1 = require("aws-cdk-lib/aws-apigateway");
@@ -6,7 +9,7 @@ const aws_iam_1 = require("aws-cdk-lib/aws-iam");
 const api_model_1 = require("../../../utils/api-model");
 const mediatypes_1 = require("../../types/mediatypes");
 const usage_plans_1 = require("../usage-plans");
-const R = require("ramda");
+const lodash_1 = __importDefault(require("lodash"));
 class DigitrafficRestApi extends aws_apigateway_1.RestApi {
     constructor(stack, apiId, apiName, allowFromIpAddresses, config) {
         const policyDocument = allowFromIpAddresses == null
@@ -58,7 +61,7 @@ class DigitrafficRestApi extends aws_apigateway_1.RestApi {
         }));
     }
     getModelWithReference(model) {
-        return R.assoc("modelReference", (0, api_model_1.getModelReference)(model.modelId, this.restApiId), model);
+        return lodash_1.default.set(model, "modelReference", (0, api_model_1.getModelReference)(model.modelId, this.restApiId));
     }
     addDocumentationPart(resource, parameterName, resourceName, type, properties) {
         const location = {

package/dist/aws/infra/stack/stack-checking-aspect.js

@@ -1,4 +1,7 @@
 "use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.StackCheckingAspect = void 0;
 const aws_cdk_lib_1 = require("aws-cdk-lib");
@@ -6,11 +9,12 @@ const aws_lambda_1 = require("aws-cdk-lib/aws-lambda");
 const aws_s3_1 = require("aws-cdk-lib/aws-s3");
 const stack_1 = require("./stack");
 const aws_apigateway_1 = require("aws-cdk-lib/aws-apigateway");
-const change_case_1 = require("change-case");
 const aws_sqs_1 = require("aws-cdk-lib/aws-sqs");
 const aws_logs_1 = require("aws-cdk-lib/aws-logs");
+const change_case_1 = require("change-case");
+const lodash_1 = __importDefault(require("lodash"));
 const MAX_CONCURRENCY_LIMIT = 100;
-const NODE_RUNTIMES = [aws_lambda_1.Runtime.NODEJS_14_X.name, aws_lambda_1.Runtime.NODEJS_16_X.name];
+const NODE_RUNTIMES = [aws_lambda_1.Runtime.NODEJS_16_X.name, aws_lambda_1.Runtime.NODEJS_18_X.name];
 var ResourceType;
 (function (ResourceType) {
     ResourceType["stackName"] = "STACK_NAME";
@@ -130,7 +134,7 @@ class StackCheckingAspect {
         return (0, change_case_1.paramCase)(path) === path;
     }
     static isValidQueryString(name) {
-        return (0, change_case_1.snakeCase)(name) === name;
+        return lodash_1.default.snakeCase(name) === name;
     }
     checkResourceCasing(node) {
         if (node instanceof aws_apigateway_1.CfnResource) {

package/dist/aws/runtime/secrets/secret-holder.js

@@ -4,6 +4,7 @@ exports.SecretHolder = void 0;
 const secret_1 = require("./secret");
 const dbsecret_1 = require("./dbsecret");
 const utils_1 = require("../../../utils/utils");
+const dt_logger_default_1 = require("../dt-logger-default");
 // eslint-disable-next-line @typescript-eslint/no-var-requires, @typescript-eslint/no-unsafe-assignment
 const NodeTtl = require("node-ttl");
 const DEFAULT_PREFIX = "";
@@ -33,7 +34,10 @@ class SecretHolder {
     }
     async initSecret() {
         const secretValue = await (0, secret_1.getSecret)(this.secretId);
-        console.info("refreshing secret " + this.secretId);
+        dt_logger_default_1.logger.info({
+            method: "SecretHolder.initSecret",
+            message: "Refreshing secret " + this.secretId,
+        });
         // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
         this.secretCache.push(DEFAULT_SECRET_KEY, secretValue);
     }

package/dist/database/database.js

@@ -2,6 +2,8 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.inDatabaseReadonly = exports.inDatabase = exports.inTransaction = exports.initDbConnection = exports.DatabaseEnvironmentKeys = void 0;
 const utils_1 = require("../utils/utils");
+const dt_logger_default_1 = require("../aws/runtime/dt-logger-default");
+const logging_1 = require("../utils/logging");
 var DatabaseEnvironmentKeys;
 (function (DatabaseEnvironmentKeys) {
     DatabaseEnvironmentKeys["DB_USER"] = "DB_USER";
@@ -68,7 +70,7 @@ async function doInDatabase(readonly, fn) {
         return await fn(db);
     }
     catch (e) {
-        console.error("Error in db:", e);
+        (0, logging_1.logException)(dt_logger_default_1.logger, e);
         throw e;
     }
     finally {

package/dist/utils/geometry.js

@@ -25,6 +25,7 @@ var __importStar = (this && this.__importStar) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.positionToList = exports.createGmlLineString = exports.distanceBetweenPositionsInM = exports.areDistinctPositions = exports.distanceBetweenPositionsInKm = exports.isFeatureCollection = exports.isValidGeoJson = exports.createFeatureCollection = exports.createGeometry = exports.SRID_WGS84 = void 0;
 const geoJsonValidator = __importStar(require("geojson-validation"));
+const dt_logger_default_1 = require("../aws/runtime/dt-logger-default");
 exports.SRID_WGS84 = 4326;
 /**
  * Creates WKT geometry from GeoJSON geometry
@@ -47,7 +48,10 @@ function createGeometry(geometry) {
         const coordinates = multiPolygon(geometry.coordinates);
         return `MULTIPOLYGON(${coordinates})`;
     }
-    console.error("unsupported locationType=%s", geometry.type);
+    dt_logger_default_1.logger.error({
+        method: "Geometry.createGeometry",
+        message: "Unsupported locationType " + geometry.type,
+    });
     return "POLYGON EMPTY";
 }
 exports.createGeometry = createGeometry;

package/dist/utils/slack.js

@@ -5,19 +5,24 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.SlackApi = void 0;
 const axios_1 = __importDefault(require("axios"));
+const dt_logger_default_1 = require("../aws/runtime/dt-logger-default");
+const logging_1 = require("./logging");
 class SlackApi {
     constructor(url) {
         this.url = url;
     }
     async notify(text) {
         try {
-            console.info('method=notify Notifying Slack');
+            dt_logger_default_1.logger.info({
+                method: "SlackApi.notify",
+                message: "Sending slack notification",
+            });
             await axios_1.default.post(this.url, {
                 text,
             });
         }
         catch (error) {
-            console.error('method=notify Slack notify failed!');
+            (0, logging_1.logException)(dt_logger_default_1.logger, error);
         }
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@digitraffic/common",
-  "version": "2023.10.11-1",
+  "version": "2023.11.21-1",
   "description": "",
   "repository": {
     "type": "git",
@@ -17,58 +17,54 @@
     "src/**/*.ts"
   ],
   "peerDependencies": {
-    "@aws-cdk/aws-synthetics-alpha": "^2.85.0-alpha.0",
-    "@types/geojson": "^7946.0.10",
-    "aws-cdk-lib": "^2.85.0",
-    "aws-sdk": "^2.1405.0",
-    "axios": "^1.2.6",
-    "change-case": "^4.1.2",
-    "constructs": "^10.2.17",
+    "@types/geojson": "^7946.0.12",
+    "aws-cdk-lib": "^2.103.0",
+    "aws-sdk": "^2.1481.0",
+    "axios": "^1.5.1",
+    "change-case": "4.1.2",
+    "constructs": "^10.3.0",
     "date-fns-tz": "~2.0.0",
     "date-fns": "~2.30.0",
     "etag": "^1.8.1",
     "geojson-validation": "^1.0.2",
     "node-ttl": "^0.2.0",
     "pg-native": "^3.0.1",
-    "pg-promise": "^11.5.0"
+    "pg-promise": "^11.5.4"
   },
   "devDependencies": {
-    "@aws-cdk/aws-synthetics-alpha": "2.85.0-alpha.0",
-    "@types/aws-lambda": "~8.10.119",
-    "@types/geojson": "^7946.0.10",
-    "@types/etag": "^1.8.1",
-    "@types/jest": "^29.5.2",
-    "@types/lodash": "^4.14.195",
-    "@types/node": "18.15.13",
-    "@types/ramda": "~0.29.1",
-    "@types/sinon": "10.0.15",
-    "@typescript-eslint/eslint-plugin": "~5.60.1",
-    "@typescript-eslint/parser": "^5.60.1",
-    "aws-cdk-lib": "~2.85.0",
-    "aws-sdk": "~2.1405.0",
-    "axios": "^1.3.6",
-    "change-case": "^4.1.2",
-    "constructs": "10.2.61",
+    "@types/aws-lambda": "8.10.125",
+    "@types/geojson": "7946.0.12",
+    "@types/etag": "1.8.2",
+    "@types/jest": "29.5.6",
+    "@types/lodash": "4.14.200",
+    "@types/node": "20.8.9",
+    "@types/sinon": "10.0.20",
+    "@typescript-eslint/eslint-plugin": "~6.9.0",
+    "@typescript-eslint/parser": "^6.9.0",
+    "aws-cdk-lib": "~2.103.1",
+    "aws-sdk": "~2.1482.0",
+    "axios": "^1.6.0",
+    "change-case": "4.1.2",
+    "constructs": "10.3.0",
     "date-fns-tz": "~2.0.0",
     "date-fns": "~2.30.0",
-    "eslint": "~8.43.0",
-    "eslint-config-prettier": "^8.8.0",
-    "eslint-plugin-deprecation": "~1.4.1",
+    "eslint": "~8.52.0",
+    "eslint-config-prettier": "^9.0.0",
+    "eslint-plugin-deprecation": "~2.0.0",
     "etag": "^1.8.1",
     "geojson-validation": "^1.0.2",
     "husky": "^8.0.3",
-    "jest": "^29.5.0",
+    "jest": "^29.7.0",
     "jest-junit": "^16.0.0",
-    "lint-staged": "^13.2.2",
-    "lodash": "^4.17.21",
+    "lint-staged": "^15.0.2",
+    "lodash": "~4.17.21",
     "node-ttl": "^0.2.0",
     "pg-native": "^3.0.1",
-    "pg-promise": "^11.5.0",
+    "pg-promise": "^11.5.4",
     "prettier": "^2.8.8",
-    "ramda": "~0.29.0",
-    "rimraf": "^5.0.1",
-    "sinon": "15.2.0",
-    "ts-jest": "^29.1.0",
+    "rimraf": "^5.0.5",
+    "sinon": "17.0.0",
+    "ts-jest": "^29.1.1",
     "typescript": "~4.9.5",
     "velocityjs": "2.0.6"
   },

package/src/aws/infra/api/integration.ts

@@ -19,7 +19,7 @@ interface ApiParameter {
     name: string;
 }
 
-export class DigitrafficIntegration {
+export class DigitrafficIntegration<T extends string> {
     readonly lambda: IFunction;
     readonly mediaType: MediaType;
     readonly parameters: ApiParameter[] = [];
@@ -35,20 +35,20 @@ export class DigitrafficIntegration {
         this.sunset = sunset;
     }
 
-    addPathParameter(...names: string[]): this {
+    addPathParameter(...names: T[]): this {
         names.forEach((name) => this.parameters.push({ type: "path", name }));
 
         return this;
     }
 
-    addQueryParameter(...names: string[]): this {
+    addQueryParameter(...names: T[]): this {
         names.forEach((name) =>
             this.parameters.push({ type: "querystring", name })
         );
         return this;
     }
 
-    addMultiValueQueryParameter(...names: string[]): this {
+    addMultiValueQueryParameter(...names: T[]): this {
         names.forEach((name) =>
             this.parameters.push({ type: "multivaluequerystring", name })
         );
@@ -61,7 +61,7 @@ export class DigitrafficIntegration {
      * @param names for the parameters
      * @returns
      */
-    addContextParameter(...names: string[]): this {
+    addContextParameter(...names: T[]): this {
         names.forEach((name) =>
             this.parameters.push({ type: "context", name })
         );
@@ -74,7 +74,7 @@ export class DigitrafficIntegration {
      * If Authorization header is needed, use lambda authorizers.
      * @param names for the headers
      */
-    addHeaderParameter(...names: string[]): this {
+    addHeaderParameter(...names: T[]): this {
         names.forEach((name) => this.parameters.push({ type: "header", name }));
 
         return this;

package/src/aws/infra/canaries/canary-alarm.ts

@@ -1,7 +1,7 @@
 import { Construct } from "constructs";
 import { CanaryParameters } from "./canary-parameters";
 import { Alarm, ComparisonOperator } from "aws-cdk-lib/aws-cloudwatch";
-import { Canary } from "@aws-cdk/aws-synthetics-alpha";
+import { Canary } from "aws-cdk-lib/aws-synthetics";
 import { SnsAction } from "aws-cdk-lib/aws-cloudwatch-actions";
 import { Topic } from "aws-cdk-lib/aws-sns";
 

package/src/aws/infra/canaries/canary-parameters.ts

@@ -1,4 +1,4 @@
-import { Schedule } from "@aws-cdk/aws-synthetics-alpha";
+import { Schedule } from "aws-cdk-lib/aws-synthetics";
 
 /** Optional env parameters for canary */
 type CanaryEnv = Record<string, string>;

package/src/aws/infra/canaries/canary.ts

@@ -5,7 +5,7 @@ import {
     Runtime,
     Schedule,
     Test,
-} from "@aws-cdk/aws-synthetics-alpha";
+} from "aws-cdk-lib/aws-synthetics";
 import { Role } from "aws-cdk-lib/aws-iam";
 import { CanaryAlarm } from "./canary-alarm";
 import { CanaryParameters } from "./canary-parameters";

package/src/aws/infra/canaries/database-checker.ts

@@ -3,6 +3,7 @@ import { ProxyHolder } from "../../runtime/secrets/proxy-holder";
 import { RdsHolder } from "../../runtime/secrets/rds-holder";
 import { getEnvVariable } from "../../../utils/utils";
 import { Countable } from "../../../database/models";
+import { logger } from "../../runtime/dt-logger-default";
 
 // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-var-requires
 const synthetics = require("Synthetics");
@@ -141,7 +142,10 @@ export class DatabaseCountChecker {
         await this.credentialsFunction();
         await inDatabaseReadonly(async (db: DTDatabase) => {
             for (const check of this.checks) {
-                console.info("canary checking sql " + check.sql);
+                logger.info({
+                    method: "DatabaseCountChecker.expect",
+                    message: "Running sql: " + check.sql,
+                });
 
                 const value = await db.one<Countable>(check.sql);
                 const checkFunction = () => {

package/src/aws/infra/canaries/url-checker.ts

@@ -10,6 +10,8 @@ import { FeatureCollection } from "geojson";
 import { isValidGeoJson } from "../../../utils/geometry";
 import { getEnvVariable } from "../../../utils/utils";
 import { ENV_API_KEY, ENV_HOSTNAME } from "./canary-keys";
+import { logger } from "../../runtime/dt-logger-default";
+import { logException } from "../../../utils/logging";
 
 export const API_KEY_HEADER = "x-api-key";
 
@@ -142,7 +144,10 @@ export class UrlChecker {
             !this.requestOptions.headers ||
             !this.requestOptions.headers[API_KEY_HEADER]
         ) {
-            console.error("No api key defined");
+            logger.error({
+                method: "UrlChecker.expect403WithoutApiKey",
+                message: "No Api-key defined",
+            });
         }
 
         const requestOptions = {
@@ -176,7 +181,7 @@ async function getResponseBody(response: IncomingMessage): Promise<string> {
         try {
             return zlib.gunzipSync(body).toString();
         } catch (e) {
-            console.info("error " + JSON.stringify(e));
+            logException(logger, e);
         }
     }
 

package/src/aws/infra/stack/monitoredfunction.ts

@@ -9,10 +9,11 @@ import {
     LambdaEnvironment,
     MonitoredFunctionParameters,
 } from "../stack/lambda-configs";
-import { pascalCase } from "change-case";
 import { DigitrafficLogSubscriptions } from "../stack/subscription";
 import { TrafficType } from "../../../types/traffictype";
 
+import _ from "lodash";
+
 /**
  * Allows customization of CloudWatch Alarm properties
  */
@@ -117,7 +118,11 @@ export class MonitoredFunction extends Function {
     ): MonitoredFunction {
         const functionName =
             functionParameters?.functionName ??
-            `${stack.configuration.shortName}-${pascalCase(name)}`;
+            `${stack.configuration.shortName}-${_.chain(name)
+                .camelCase()
+                .startCase()
+                .replace(/\s/g, "")
+                .value()}`;
         const functionProps = databaseFunctionProps(
             stack,
             environment,
@@ -315,7 +320,11 @@ export class MonitoredDBFunction {
     ): MonitoredFunction {
         const functionName =
             functionParameters?.functionName ??
-            `${stack.configuration.shortName}-${pascalCase(name)}`;
+            `${stack.configuration.shortName}-${_.chain(name)
+                .camelCase()
+                .startCase()
+                .replace(/\s/g, "")
+                .value()}`;
         const env = environment ? environment : stack.createLambdaEnvironment();
         const functionProps = databaseFunctionProps(
             stack,

package/src/aws/infra/stack/rest_apis.ts

@@ -26,7 +26,7 @@ import { DocumentationPart, DocumentationProperties } from "../documentation";
 import { createDefaultUsagePlan, createUsagePlan } from "../usage-plans";
 import { DigitrafficStack } from "./stack";
 
-import R = require("ramda");
+import _ from "lodash";
 
 export class DigitrafficRestApi extends RestApi {
     readonly apiKeyIds: string[];
@@ -109,10 +109,10 @@ export class DigitrafficRestApi extends RestApi {
     }
 
     private getModelWithReference(model: Model): ModelWithReference {
-        return R.assoc(
+        return _.set(
+            model,
             "modelReference",
-            getModelReference(model.modelId, this.restApiId),
-            model
+            getModelReference(model.modelId, this.restApiId)
         ) as ModelWithReference;
     }
 

package/src/aws/infra/stack/stack-checking-aspect.ts

@@ -4,13 +4,14 @@ import { CfnBucket } from "aws-cdk-lib/aws-s3";
 import { DigitrafficStack, SOLUTION_KEY } from "./stack";
 import { IConstruct } from "constructs";
 import { CfnMethod, CfnResource } from "aws-cdk-lib/aws-apigateway";
-import { paramCase, snakeCase } from "change-case";
 import { CfnQueue } from "aws-cdk-lib/aws-sqs";
 import { LogRetention } from "aws-cdk-lib/aws-logs";
+import { paramCase } from "change-case";
+import _ from "lodash";
 import IntegrationProperty = CfnMethod.IntegrationProperty;
 
 const MAX_CONCURRENCY_LIMIT = 100;
-const NODE_RUNTIMES = [Runtime.NODEJS_14_X.name, Runtime.NODEJS_16_X.name];
+const NODE_RUNTIMES = [Runtime.NODEJS_16_X.name, Runtime.NODEJS_18_X.name];
 
 enum ResourceType {
     stackName = "STACK_NAME",
@@ -214,7 +215,7 @@ export class StackCheckingAspect implements IAspect {
     }
 
     private static isValidQueryString(name: string) {
-        return snakeCase(name) === name;
+        return _.snakeCase(name) === name;
     }
 
     private checkResourceCasing(node: IConstruct) {

package/src/aws/runtime/secrets/secret-holder.ts

@@ -1,6 +1,7 @@
 import { GenericSecret, getSecret } from "./secret";
 import { checkExpectedSecretKeys } from "./dbsecret";
 import { getEnvVariable } from "../../../utils/utils";
+import { logger } from "../dt-logger-default";
 
 // eslint-disable-next-line @typescript-eslint/no-var-requires, @typescript-eslint/no-unsafe-assignment
 const NodeTtl = require("node-ttl");
@@ -47,7 +48,10 @@ export class SecretHolder<Secret extends GenericSecret> {
     private async initSecret() {
         const secretValue = await getSecret<Secret>(this.secretId);
 
-        console.info("refreshing secret " + this.secretId);
+        logger.info({
+            method: "SecretHolder.initSecret",
+            message: "Refreshing secret " + this.secretId,
+        });
 
         // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
         this.secretCache.push(DEFAULT_SECRET_KEY, secretValue);

package/src/database/database.ts

@@ -1,5 +1,7 @@
 import { IDatabase, ITask } from "pg-promise";
 import { getEnvVariable, getEnvVariableOrElse } from "../utils/utils";
+import { logger } from "../aws/runtime/dt-logger-default";
+import { logException } from "../utils/logging";
 
 export enum DatabaseEnvironmentKeys {
     DB_USER = "DB_USER",
@@ -97,7 +99,8 @@ async function doInDatabase<T>(
         await db.none("DISCARD ALL");
         return await fn(db);
     } catch (e) {
-        console.error("Error in db:", e);
+        logException(logger, e);
+
         throw e;
     } finally {
         await db.$pool.end();

package/src/utils/geometry.ts

@@ -3,6 +3,7 @@
  */
 import { Feature, FeatureCollection, Geometry, Position } from "geojson";
 import * as geoJsonValidator from "geojson-validation";
+import { logger } from "../aws/runtime/dt-logger-default";
 
 export const SRID_WGS84 = 4326;
 
@@ -29,7 +30,11 @@ export function createGeometry(geometry: Geometry): string {
         return `MULTIPOLYGON(${coordinates})`;
     }
 
-    console.error("unsupported locationType=%s", geometry.type);
+    logger.error({
+        method: "Geometry.createGeometry",
+        message: "Unsupported locationType " + geometry.type,
+    });
+
     return "POLYGON EMPTY";
 }
 

package/src/utils/slack.ts

@@ -1,22 +1,26 @@
 import axios from "axios";
+import { logger } from "../aws/runtime/dt-logger-default";
+import { logException } from "./logging";
 
 export class SlackApi {
-
     private readonly url: string;
 
-    constructor(url :string) {
+    constructor(url: string) {
         this.url = url;
     }
 
     async notify(text: string) {
         try {
-            console.info('method=notify Notifying Slack');
+            logger.info({
+                method: "SlackApi.notify",
+                message: "Sending slack notification",
+            });
+
             await axios.post(this.url, {
                 text,
             });
         } catch (error) {
-            console.error('method=notify Slack notify failed!');
+            logException(logger, error);
         }
     }
-
 }