@certenza/aws-cdk-infrastructure-commons 1.1.1 → 1.1.2

package/dist/src/apigateway.js

@@ -67,6 +67,11 @@ const createApiGateway = (scope, apiName, domainName, hostedZoneId, zoneName) =>
     const hostedZone = (0, route53_1.getHostedZone)(scope, "HostedZone", hostedZoneId, zoneName);
     // Create a certificate for the API Gateway domain
     const certificate = (0, acm_1.createCertificate)(scope, `ApiGatewayCertificate`, domainName, hostedZone);
+    // Create access log group
+    const accessLogGroup = new logs.LogGroup(scope, `${apiName}-AccessLogs`, {
+        retention: logs.RetentionDays.ONE_WEEK,
+    });
+    // Create RestApi without auto-deployment so we can control the order of resource creation
     const api = new apigateway.RestApi(scope, apiName, {
         description: `Public API Gateway for ${apiName}`,
         endpointTypes: [apigateway.EndpointType.REGIONAL],
@@ -76,24 +81,37 @@ const createApiGateway = (scope, apiName, domainName, hostedZoneId, zoneName) =>
             securityPolicy: apigateway.SecurityPolicy.TLS_1_2,
             endpointType: apigateway.EndpointType.REGIONAL,
         },
-        deployOptions: {
-            loggingLevel: apigateway.MethodLoggingLevel.INFO,
-            dataTraceEnabled: false,
-            accessLogDestination: new apigateway.LogGroupLogDestination(new logs.LogGroup(scope, `${apiName}-AccessLogs`, {
-                retention: logs.RetentionDays.ONE_WEEK,
-            })),
-        },
+        deploy: false, // Don't auto-deploy - we'll create deployment and stage manually
     });
-    // Set retention for execution log group that API Gateway creates automatically
+    // Create execution log group with the exact name API Gateway expects
     // API Gateway creates execution logs in the format: API-Gateway-Execution-Logs_{api-id}/{stage-name}
-    // We use LogRetention to set the retention policy after API Gateway creates the log group
-    new logs.LogRetention(scope, `${apiName}-ExecutionLogRetention`, {
+    // By creating it first, API Gateway will use our log group instead of creating a new one
+    const executionLogGroup = new logs.LogGroup(scope, `${apiName}-ExecutionLogs`, {
         logGroupName: cdk.Token.asString(cdk.Fn.join("/", [
-            cdk.Fn.join("_", ["API-Gateway-Execution-Logs", api.restApiId]),
-            api.deploymentStage.stageName,
+            cdk.Fn.join("_", [
+                "API-Gateway-Execution-Logs",
+                api.restApiId,
+            ]),
+            "prod", // Default stage name
         ])),
         retention: logs.RetentionDays.ONE_MONTH, // Budget-friendly: 1 month retention
+        removalPolicy: cdk.RemovalPolicy.DESTROY, // Retain log group even if API is deleted
+    });
+    // Create deployment
+    const deployment = new apigateway.Deployment(scope, `${apiName}-Deployment`, {
+        api: api,
+    });
+    // Create stage with logging configuration
+    // The execution log group must exist before the stage is created
+    const stage = new apigateway.Stage(scope, `${apiName}-Stage`, {
+        deployment: deployment,
+        stageName: "prod",
+        loggingLevel: apigateway.MethodLoggingLevel.INFO,
+        dataTraceEnabled: false,
+        accessLogDestination: new apigateway.LogGroupLogDestination(accessLogGroup),
     });
+    // Ensure the execution log group is created before the stage
+    stage.node.addDependency(executionLogGroup);
     // Create Route53 A record pointing to the API Gateway
     new route53.ARecord(scope, `${apiName}-AliasRecord`, {
         zone: hostedZone,

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@certenza/aws-cdk-infrastructure-commons",
-    "version": "1.1.1",
+    "version": "1.1.2",
     "description": "Common infrastructure reusable utilities and resources for Certenza projects",
     "main": "dist/index.js",
     "types": "dist/index.d.ts",