Merge pull request #188 from DataDog/fix/shared-cdk

fix: resolve issues with DNS parsing

Author: jeastham1993

shared/infra/aws/lib/dns.ts

@@ -34,19 +34,20 @@ export class Dns extends Construct {
         "ImportedCertificate",
         certificateArn,
       );
+
     } else {
       this.certificate = undefined;
       this.hostedZone = undefined;
     }
   }
 
-  addCnameFor(distribution: Distribution) {
+  addCnameFor(distribution: Distribution, env: string) {
     // Add a CName if the hosted zone exists.
     if (this.hostedZone) {
       const cNameRecord = new CnameRecord(this, "CnameRecord", {
         zone: this.hostedZone!,
         domainName: distribution.domainName,
-        recordName: this.getPrimaryDomainName("prod")!,
+        recordName: this.getPrimaryDomainName(env)!,
         ttl: Duration.minutes(5),
       });
     }

shared/infra/aws/lib/network.ts

@@ -161,7 +161,7 @@ export class Network extends Construct {
       comment: `OAI for stickerlandia web frontend ${props.env}`,
     });
     webFrontendBucket.grantRead(originIdentity);
-
+    
     this.distribution = new Distribution(this, `Stickerlandia-${props.env}`, {
       certificate: props.dns.certificate,
       domainNames: props.dns.getPrimaryDomainName(props.env)

shared/infra/aws/lib/shared-resources-stack.ts

@@ -31,7 +31,7 @@ export class StickerlandiaSharedResourcesStack extends cdk.Stack {
       dns: dns,
     });
 
-    dns.addCnameFor(network.distribution);
+    dns.addCnameFor(network.distribution, env);
 
     const persistence = new Persistence(this, "Persistence", {
       env,