@digitraffic/common 2022.12.22-1 → 2022.12.22-3

package/dist/aws/infra/api/handler-factory.d.ts

@@ -1,7 +1,7 @@
 import { DtLogger } from "../../runtime/dt-logger";
 import { LambdaResponse } from "../../types/lambda-response";
 export type LoggingHandler = (method: () => Promise<LambdaResponse>, logger: DtLogger) => Promise<LambdaResponse>;
-export type ErrorHandler = (error: unknown) => LambdaResponse;
+export type ErrorHandler = (error: unknown, logger: DtLogger) => LambdaResponse;
 /**
  * Factory class for creating lambda-handler functions.  You can set functionality to handle logging and error-handling,
  * with the defaults:

package/dist/aws/infra/api/handler-factory.js

@@ -12,7 +12,15 @@ exports.createJsonLoggingHandler = exports.HandlerFactory = void 0;
  */
 class HandlerFactory {
     constructor() {
-        this.loggingHandler = createDefaultLoggingHandler();
+        this.loggingHandler = async (method) => {
+            const start = Date.now();
+            try {
+                return await method();
+            }
+            finally {
+                console.info("method=%s.handler tookMs=%d", process.env.AWS_LAMBDA_FUNCTION_NAME, Date.now() - start);
+            }
+        };
         this.errorHandler = (error) => {
             throw error;
         };
@@ -32,24 +40,13 @@ class HandlerFactory {
                     return await handler(event);
                 }
                 catch (error) {
-                    return this.errorHandler(error);
+                    return this.errorHandler(error, logger);
                 }
             }, logger);
         };
     }
 }
 exports.HandlerFactory = HandlerFactory;
-function createDefaultLoggingHandler() {
-    return async (method) => {
-        const start = Date.now();
-        try {
-            return await method();
-        }
-        finally {
-            console.info("method=%s.handler tookMs=%d", process.env.AWS_LAMBDA_FUNCTION_NAME, Date.now() - start);
-        }
-    };
-}
 function createJsonLoggingHandler() {
     return async (method, logger) => {
         const start = Date.now();

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

@@ -110,10 +110,10 @@ class StackCheckingAspect {
     checkBucket(node) {
         if (node instanceof aws_s3_1.CfnBucket) {
             const c = node.publicAccessBlockConfiguration;
-            if (!c.blockPublicAcls ||
+            if (c && (!c.blockPublicAcls ||
                 !c.blockPublicPolicy ||
                 !c.ignorePublicAcls ||
-                !c.restrictPublicBuckets) {
+                !c.restrictPublicBuckets)) {
                 this.addAnnotation(node, ResourceType.bucketPublicity, "Check bucket publicity");
             }
         }

package/dist/aws/infra/stacks/db-stack.js

@@ -38,7 +38,8 @@ class DbStack extends aws_cdk_lib_1.Stack {
                 }),
                 parameters: {
                     "pg_stat_statements.track": "ALL",
-                    random_page_cost: "1",
+                    "random_page_cost": "1",
+                    "work_mem": "512MB"
                 },
             })
             : aws_rds_1.ParameterGroup.fromParameterGroupName(this, "ParameterGroup", `default.aurora-postgresql${configuration.dbVersion.auroraPostgresMajorVersion}`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@digitraffic/common",
-  "version": "2022.12.22-1",
+  "version": "2022.12.22-3",
   "description": "",
   "repository": {
     "type": "git",

package/src/aws/infra/api/handler-factory.ts

@@ -6,7 +6,7 @@ export type LoggingHandler = (
     logger: DtLogger
 ) => Promise<LambdaResponse>;
 
-export type ErrorHandler = (error: unknown) => LambdaResponse;
+export type ErrorHandler = (error: unknown, logger: DtLogger) => LambdaResponse;
 
 /**
  * Factory class for creating lambda-handler functions.  You can set functionality to handle logging and error-handling,
@@ -22,7 +22,19 @@ export class HandlerFactory {
     private errorHandler: ErrorHandler;
 
     constructor() {
-        this.loggingHandler = createDefaultLoggingHandler();
+        this.loggingHandler = async (method: () => Promise<LambdaResponse>) => {
+            const start = Date.now();
+
+            try {
+                return await method();
+            } finally {
+                console.info(
+                    "method=%s.handler tookMs=%d",
+                    process.env.AWS_LAMBDA_FUNCTION_NAME,
+                    Date.now() - start
+                );
+            }
+        };
 
         this.errorHandler = (error: unknown) => {
             throw error;
@@ -48,29 +60,13 @@ export class HandlerFactory {
                 try {
                     return await handler(event);
                 } catch (error) {
-                    return this.errorHandler(error);
+                    return this.errorHandler(error, logger);
                 }
             }, logger);
         };
     }
 }
 
-function createDefaultLoggingHandler(): LoggingHandler {
-    return async (method: () => Promise<LambdaResponse>) => {
-        const start = Date.now();
-
-        try {
-            return await method();
-        } finally {
-            console.info(
-                "method=%s.handler tookMs=%d",
-                process.env.AWS_LAMBDA_FUNCTION_NAME,
-                Date.now() - start
-            );
-        }
-    };
-}
-
 export function createJsonLoggingHandler(): LoggingHandler {
     return async (method: () => Promise<LambdaResponse>, logger: DtLogger) => {
         const start = Date.now();

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

@@ -181,14 +181,14 @@ export class StackCheckingAspect implements IAspect {
     private checkBucket(node: IConstruct) {
         if (node instanceof CfnBucket) {
             const c =
-                node.publicAccessBlockConfiguration as CfnBucket.PublicAccessBlockConfigurationProperty;
+                node.publicAccessBlockConfiguration as CfnBucket.PublicAccessBlockConfigurationProperty | undefined;
 
-            if (
+            if (c && (
                 !c.blockPublicAcls ||
                 !c.blockPublicPolicy ||
                 !c.ignorePublicAcls ||
                 !c.restrictPublicBuckets
-            ) {
+            )) {
                 this.addAnnotation(
                     node,
                     ResourceType.bucketPublicity,