biffgaut

Thanks, we'll take a look. To clarify, would you want to add additional headers to what the construct already applies for security headers or would you want to take complete...

The headers sent the cloudFrontDistributionProps will be sent to the CloudFront Distribution CDK Constructor [here](https://github.com/awslabs/aws-solutions-constructs/blob/85b5f7ada3e197dcc83a1ad1bd9e23efedf9f63e/source/patterns/%40aws-solutions-constructs/core/lib/cloudfront-distribution-helper.ts#L78-L80). If props.insertHttpSecurityHeaders is true (which is the default), then a CloudFront Function is created to...

dreamorosi@ - go ahead on everything that meets the design guidelines. Let's continue to discuss the new properties your proposing (logGroupProps and createCloudWatchAlarms) and the implications on the rest of...

dreamorosi@ - your design is correct, our design guidelines doc missed these props. I will update the DESIGN_GUIDELINES.md file - you can proceed as you defined the construct above.

One of the tenets of our design is that Constructs are consistent - once a developer uses a construct they should be able to use other constructs intuitively. While these...

Sorry for the delayed response. Our team meeting is tomorrow and we'll discuss - thanks.

This seems like an idea that would bring value to customers. A couple adjustments- * All this functionality wouldn't be published as a single construct. We see it as 2...

Yes, that's how we see the functionality divided - although route53-apigateway will probably accept an existing API generated by another construct rather than create an API from scratch, I believe...

Thanks - we'll take a look.

Thanks for the input. We're currently in the final stages of updating this construct to deploy in a VPC (non-VPC deployments will still work). In the process we found that...