Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CloudFront] Distribution - Origin Groups #9109

Closed
njlynch opened this issue Jul 16, 2020 · 0 comments · Fixed by #9360
Closed

[CloudFront] Distribution - Origin Groups #9109

njlynch opened this issue Jul 16, 2020 · 0 comments · Fixed by #9360
Assignees
Labels
@aws-cdk/aws-cloudfront Related to Amazon CloudFront effort/medium Medium work item – several days of effort feature-request A feature should be added or improved. in-progress This issue is being actively worked on.

Comments

@njlynch
Copy link
Contributor

njlynch commented Jul 16, 2020

Create support for Origin Groups (fallback origin configurations) for the new Distribution construct.

Proposed Solution

See aws/aws-cdk-rfcs#171.


This is a 🚀 Feature Request

@njlynch njlynch added feature-request A feature should be added or improved. @aws-cdk/aws-cloudfront Related to Amazon CloudFront effort/medium Medium work item – several days of effort labels Jul 16, 2020
@njlynch njlynch self-assigned this Jul 16, 2020
@skinny85 skinny85 assigned skinny85 and unassigned njlynch Jul 24, 2020
@skinny85 skinny85 added the in-progress This issue is being actively worked on. label Jul 24, 2020
mergify bot pushed a commit that referenced this issue Jul 29, 2020
Change the Origin to an interface, from an abstract class, and change its `bind` protocol to return an `OriginBindConfig` interface.

This is in preparation for handling Origin Groups in #9109 - when time comes to handle Origin Groups, we will add a new (optional) property to `OriginBindConfig`, of type `CfnDistribution.OriginGroupProperty`, and handle it in `Distribution`.

BREAKING CHANGE: the property Origin.domainName has been removed

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
skinny85 added a commit to skinny85/aws-cdk that referenced this issue Jul 30, 2020
skinny85 added a commit to skinny85/aws-cdk that referenced this issue Jul 30, 2020
skinny85 added a commit to skinny85/aws-cdk that referenced this issue Jul 31, 2020
skinny85 added a commit to skinny85/aws-cdk that referenced this issue Aug 3, 2020
@mergify mergify bot closed this as completed in #9360 Aug 3, 2020
mergify bot pushed a commit that referenced this issue Aug 3, 2020
Fixes #9109

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
eladb pushed a commit that referenced this issue Aug 10, 2020
Fixes #9109

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
curtiseppel pushed a commit to curtiseppel/aws-cdk that referenced this issue Aug 11, 2020
Change the Origin to an interface, from an abstract class, and change its `bind` protocol to return an `OriginBindConfig` interface.

This is in preparation for handling Origin Groups in aws#9109 - when time comes to handle Origin Groups, we will add a new (optional) property to `OriginBindConfig`, of type `CfnDistribution.OriginGroupProperty`, and handle it in `Distribution`.

BREAKING CHANGE: the property Origin.domainName has been removed

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
curtiseppel pushed a commit to curtiseppel/aws-cdk that referenced this issue Aug 11, 2020
Fixes aws#9109

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@aws-cdk/aws-cloudfront Related to Amazon CloudFront effort/medium Medium work item – several days of effort feature-request A feature should be added or improved. in-progress This issue is being actively worked on.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants