From 366c7819321dfd23d4ce638fb218fe38072e90aa Mon Sep 17 00:00:00 2001 From: Nick Lynch Date: Wed, 19 Aug 2020 20:02:04 +0100 Subject: [PATCH] fix(cloudfront): Distribution ignores webAclId (#9828) This was missed in #9635; the property was added to DistributionProps but not included in the underlying CfnDistribution. fixes #9824 ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license* --- packages/@aws-cdk/aws-cloudfront/lib/distribution.ts | 1 + packages/@aws-cdk/aws-cloudfront/test/distribution.test.ts | 1 + 2 files changed, 2 insertions(+) diff --git a/packages/@aws-cdk/aws-cloudfront/lib/distribution.ts b/packages/@aws-cdk/aws-cloudfront/lib/distribution.ts index 42032f547563c..a53d38fb054be 100644 --- a/packages/@aws-cdk/aws-cloudfront/lib/distribution.ts +++ b/packages/@aws-cdk/aws-cloudfront/lib/distribution.ts @@ -284,6 +284,7 @@ export class Distribution extends Resource implements IDistribution { priceClass: props.priceClass ?? undefined, restrictions: this.renderRestrictions(props.geoRestriction), viewerCertificate: this.certificate ? this.renderViewerCertificate(this.certificate) : undefined, + webAclId: props.webAclId, }, }); diff --git a/packages/@aws-cdk/aws-cloudfront/test/distribution.test.ts b/packages/@aws-cdk/aws-cloudfront/test/distribution.test.ts index ff629ade249c4..0d79aa18bf10b 100644 --- a/packages/@aws-cdk/aws-cloudfront/test/distribution.test.ts +++ b/packages/@aws-cdk/aws-cloudfront/test/distribution.test.ts @@ -100,6 +100,7 @@ test('exhaustive example of props renders correctly', () => { SslSupportMethod: 'sni-only', MinimumProtocolVersion: 'TLSv1.2_2019', }, + WebACLId: '473e64fd-f30b-4765-81a0-62ad96dd167a', }, }); });