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

New module for creating Cloudfront header policies… #925

Conversation

stefanhorning
Copy link
Contributor

.. used for response headers

SUMMARY

New Cloudfront module for CF response headers policies, see https://docs.aws.amazon.com/AmazonCloudFront/latest/DeveloperGuide/adding-response-headers.html

This is still a relavily new feature, see https://aws.amazon.com/de/blogs/networking-and-content-delivery/amazon-cloudfront-introduces-response-headers-policies/

ISSUE TYPE
  • New Module Pull Request
COMPONENT NAME

cloudfront_response_headers_policy.py

@ansibullbot
Copy link

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (third-party-check pipeline).

@stefanhorning
Copy link
Contributor Author

still needs tests and perhaps a changelog fragment

@softwarefactory-project-zuul
Copy link
Contributor

@marknet15
Copy link
Contributor

marknet15 commented Feb 4, 2022

@stefanhorning no changelog needed for a new module (as it auto detects new modules on releases) but it does need adding to here I believe:
https://github.com/ansible-collections/community.aws/blob/main/meta/runtime.yml

@ansibullbot ansibullbot added integration tests/integration tests tests labels Feb 7, 2022
@stefanhorning
Copy link
Contributor Author

Build green and ready to merge.

@stefanhorning
Copy link
Contributor Author

stefanhorning commented Feb 18, 2022

@alinabuzachis Thanks for the review. I incorporated your suggestions now.

@markuman
Copy link
Member

upstream CI error...

@markuman markuman added the backport-3 PR should be backported to the stable-3 branch label Feb 22, 2022
@markuman
Copy link
Member

recheck

@stefanhorning
Copy link
Contributor Author

Ready to be merged now (from my side).

@markuman markuman requested a review from alinabuzachis March 14, 2022 19:34
@alinabuzachis alinabuzachis added the mergeit Merge the PR (SoftwareFactory) label Mar 15, 2022
@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 87848dc into ansible-collections:main Mar 15, 2022
@patchback
Copy link

patchback bot commented Mar 15, 2022

Backport to stable-3: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-3/87848dcc0747fa4cb3846d4aca3849d826463548/pr-925

Backported as #997

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Mar 15, 2022
New module for creating Cloudfront header policies…

.. used for response headers
SUMMARY
New Cloudfront module for CF response headers policies, see https://docs.aws.amazon.com/AmazonCloudFront/latest/DeveloperGuide/adding-response-headers.html
This is still a relavily new feature, see https://aws.amazon.com/de/blogs/networking-and-content-delivery/amazon-cloudfront-introduces-response-headers-policies/
ISSUE TYPE

New Module Pull Request

COMPONENT NAME
cloudfront_response_headers_policy.py

Reviewed-by: Mark Woolley <[email protected]>
Reviewed-by: Stefan Horning <None>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Markus Bergholz <[email protected]>
(cherry picked from commit 87848dc)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Mar 15, 2022
[PR #925/87848dcc backport][stable-3] New module for creating Cloudfront header policies…

This is a backport of PR #925 as merged into main (87848dc).
.. used for response headers
SUMMARY
New Cloudfront module for CF response headers policies, see https://docs.aws.amazon.com/AmazonCloudFront/latest/DeveloperGuide/adding-response-headers.html
This is still a relavily new feature, see https://aws.amazon.com/de/blogs/networking-and-content-delivery/amazon-cloudfront-introduces-response-headers-policies/
ISSUE TYPE

New Module Pull Request

COMPONENT NAME
cloudfront_response_headers_policy.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-3 PR should be backported to the stable-3 branch community_review integration tests/integration mergeit Merge the PR (SoftwareFactory) module module new_module New module new_plugin New plugin plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants