fix: removes the requirement to have a domain name when using SSL certificates

[nsmith-amplience]

This PR allows for domain names to be optional when creating a Cloudfront Distribution that has a certificate attached to it. As Cloudfront Distributions allow for domains to be transferred with the associate-alias command there are scenarios where having an SSL certificate attached without a domain are valid, and attaching the SSL certificate in the CDK creation phase minimises the amount of stack drift.

---

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

[aws-cdk-automation]

The pull request linter has failed. See the aws-cdk-automation comment below for failure reasons. If you believe this pull request should receive an exemption, please comment and provide a justification.

A comment requesting an exemption should contain the text Exemption Request. Additionally, if clarification is needed add Clarification Request to a comment.

[aws-cdk-automation]

AWS CodeBuild CI Report

• CodeBuild project: AutoBuildv2Project1C6BFA3F-wQm2hXv2jqQv
• Commit ID: 99a4e30
• Result: FAILED
• Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

[nsmith-amplience]

Exemption Request

This PR is rolling back a previous change rather than implementing new functionality, it doesn't seem to meet the criteria in When are Integration Tests required.

[kaizencc]

Hi @nsmith-amplience, before we can look at this PR the build needs to succeed. The current issue is:

aws-cdk-lib: aws-cloudfront/test/distribution.test.ts:514:1 - error TS1128: Declaration or statement expected.
aws-cdk-lib: 514 });
aws-cdk-lib:     ~
aws-cdk-lib: aws-cloudfront/test/distribution.test.ts:514:2 - error TS1128: Declaration or statement expected.
aws-cdk-lib: 514 });

[aws-cdk-automation]

This PR has been in the BUILD FAILING state for 3 weeks, and looks abandoned. To keep this PR from being closed, please continue work on it. If not, it will automatically be closed in a week.

[aws-cdk-automation]

This PR has been deemed to be abandoned, and will be automatically closed. Please create a new PR for these changes if you think this decision has been made in error.

[aws-cdk-automation]

The pull request linter fails with the following errors:

❌ Fixes must contain a change to an integration test file and the resulting snapshot.

PRs must pass status checks before we can provide a meaningful review.

If you would like to request an exemption from the status checks or clarification on feedback, please leave a comment on this PR containing Exemption Request and/or Clarification Request.