-
Notifications
You must be signed in to change notification settings - Fork 741
Import Cloudfront PQ TLS Policies #5539
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The other policies without "no-pq" contain PQ. Does this one intentionally not have PQ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this policy is a legacy used for testing, hence not upgrading to PQ
/* CloudFront viewer facing */ | ||
"CloudFront-SSL-v-3", | ||
"CloudFront-TLS-1-0-2014", | ||
"CloudFront-TLS-1-0-2014-PQ-Beta", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aren't there a bunch of other new policies that support TLS 1.3?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
most of the viewer side policies use TLS 1.3, you might be thinking about origin policies that have a separate policies that have TLS 1.3
Are there any CloudFront policies that could be deleted? If the answer is no that's fine, but it this seems like a good point to check in on that 🙂 |
Release Summary:
Resolved issues:
P302735809
Description of changes:
Adds CloudFront PQ TLS Policies
Call-outs:
None
Testing:
Unit Tests
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.