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

aws-cloudfront-s3 README issue and cannot generate OAI resource #127

Closed
JoeShi opened this issue Feb 2, 2021 · 4 comments
Closed

aws-cloudfront-s3 README issue and cannot generate OAI resource #127

JoeShi opened this issue Feb 2, 2021 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@JoeShi
Copy link

JoeShi commented Feb 2, 2021

  1. The README documentation of aws-cloudfront-s3 pattern is not correct.
  2. The pattern did not create OAI anymore. I remember in early version, it do create the CloudFront OAI. Example can be seen here.

Reproduction Steps

In the documentation here. The type of cloudFrontDistributionProps is cloudfront.CloudFrontWebDistributionProps. However, in the code, it is cloudfront.DistributionProps.

Error Log

No.

Environment

  • CDK CLI Version :
  • CDK Framework Version:
  • AWS Solutions Constructs Version :
  • OS :
  • Language :

Other


This is 🐛 Bug Report

@JoeShi JoeShi added bug Something isn't working needs-triage The issue or PR still needs to be triaged labels Feb 2, 2021
@JoeShi JoeShi changed the title aws0c aws-cloudfront-s3 README issue and cannot generate AOI resource Feb 2, 2021
@JoeShi JoeShi changed the title aws-cloudfront-s3 README issue and cannot generate AOI resource aws-cloudfront-s3 README issue and cannot generate OAI resource Feb 2, 2021
@biffgaut
Copy link
Contributor

biffgaut commented Feb 2, 2021

Thanks - we'll take a look and see what's going on.

@hnishar
Copy link
Contributor

hnishar commented Feb 2, 2021

Thanks for the feedback. Since v1.68.0, for all Cloudfront patterns, the underlying Cloudfront L2 CDK construct has changed from CloudFrontWebDistribution to Distribution See the Changelog and related issue We will update the READMEs

@hnishar
Copy link
Contributor

hnishar commented Feb 2, 2021

One of the benefits of using the new L2 CDK construct was that it creates the OAI hence the aws-cloudfront-s3 pattern no longer needs to create the OAI, so if you change your input props from CloudFrontWebDistributionProps to DistributionProps you should see the OAI resource being created.

@hnishar hnishar self-assigned this Feb 4, 2021
@hnishar hnishar removed the needs-triage The issue or PR still needs to be triaged label Feb 4, 2021
@hnishar
Copy link
Contributor

hnishar commented Feb 17, 2021

JoeShi@ Documentation has been updated to reflect the updated Pattern Construct Props for aws-cloudfront-s3 pattern

https://docs.aws.amazon.com/solutions/latest/constructs/aws-cloudfront-s3.html#pattern-construct-props-5

@hnishar hnishar closed this as completed Feb 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants