From 3433cd6f0e44457bb665c3f92b5f4fae1e7df104 Mon Sep 17 00:00:00 2001 From: Kikuo Emoto Date: Sat, 24 Sep 2022 13:58:17 +0900 Subject: [PATCH] chore(cdk-ops): rename AccessLogsMasking MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - `AccessLogsMasking` → `AccessLogsETL`. Because the construct will not just mask access logs but do more transformation. So Extract, Transform, and Load (ETL) should be more suitable. issue codemonger-io/codemonger#30 --- .../lib/{access-logs-masking.ts => access-logs-etl.ts} | 10 ++++++++-- cdk-ops/lib/cdk-ops-stack.ts | 6 +++--- 2 files changed, 11 insertions(+), 5 deletions(-) rename cdk-ops/lib/{access-logs-masking.ts => access-logs-etl.ts} (96%) diff --git a/cdk-ops/lib/access-logs-masking.ts b/cdk-ops/lib/access-logs-etl.ts similarity index 96% rename from cdk-ops/lib/access-logs-masking.ts rename to cdk-ops/lib/access-logs-etl.ts index 39b80fc..e867396 100644 --- a/cdk-ops/lib/access-logs-masking.ts +++ b/cdk-ops/lib/access-logs-etl.ts @@ -21,8 +21,14 @@ export interface Props { deploymentStage: DeploymentStage; } -/** CDK construct that provisions resources to mask CloudFront access logs. */ -export class AccessLogsMasking extends Construct { +/** + * CDK construct that provisions resources to process CloudFront access logs. + * + * @remarks + * + * Defines extract, transform, and load (ETL) operations. + */ +export class AccessLogsETL extends Construct { /** S3 bucket for masked access logs. */ readonly maskedAccessLogsBucket: s3.IBucket; diff --git a/cdk-ops/lib/cdk-ops-stack.ts b/cdk-ops/lib/cdk-ops-stack.ts index a7bae06..135e4ed 100644 --- a/cdk-ops/lib/cdk-ops-stack.ts +++ b/cdk-ops/lib/cdk-ops-stack.ts @@ -1,7 +1,7 @@ import { Stack, StackProps } from 'aws-cdk-lib'; import { Construct } from 'constructs'; -import { AccessLogsMasking } from './access-logs-masking'; +import { AccessLogsETL } from './access-logs-etl'; import { CodemongerResources, CodemongerResourceNames, @@ -25,9 +25,9 @@ export class CdkOpsStack extends Stack { const pipeline = new ContentsPipeline(this, 'ContentsPipeline', { codemongerResources, }); - const developmentContentsAccessLogsMasking = new AccessLogsMasking( + const developmentContentsAccessLogsETL = new AccessLogsETL( this, - 'DevelopmentContentsAccessLogsMasking', + 'DevelopmentContentsAccessLogsETL', { accessLogsBucket: codemongerResources.developmentContentsAccessLogsBucket,