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

feat: amplify custom headers configuration #46

Merged
merged 2 commits into from
Jan 21, 2025
Merged

Conversation

oycyc
Copy link
Contributor

@oycyc oycyc commented Jan 18, 2025

what

Allow setting custom headers in Amplify

https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/amplify_app#custom-headers
From Terraform docs:
image

why

As said in the AWS Amplify docs:
"Custom HTTP headers enable you to specify headers for every HTTP response. Response headers can be used for debugging, security, and informational purposes. You can specify headers in the Amplify console, or by downloading and editing an app's customHttp.yml file and saving it in the project's root directory."

https://docs.aws.amazon.com/amplify/latest/userguide/custom-headers.html

references

https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/amplify_app#custom-headers

@oycyc oycyc requested review from a team as code owners January 18, 2025 16:20
@oycyc oycyc requested review from hans-d and jamengual January 18, 2025 16:20
@mergify mergify bot added the triage Needs triage label Jan 18, 2025
@RoseSecurity
Copy link
Contributor

/terratest

@RoseSecurity
Copy link
Contributor

@oycyc Thank you for this contribution! The tests are currently failing for an unrelated reason that we are working to resolve. Thank you for opening this PR and know that we are actively working to get it merged

@oycyc
Copy link
Contributor Author

oycyc commented Jan 21, 2025

@oycyc Thank you for this contribution! The tests are currently failing for an unrelated reason that we are working to resolve. Thank you for opening this PR and know that we are actively working to get it merged

ty for the quick response!

@oycyc
Copy link
Contributor Author

oycyc commented Jan 21, 2025

Ah
image

I'd say just remove Terratest and use vanilla Terraform framework tests 😆 but that'll be a huge effort

@goruha
Copy link
Member

goruha commented Jan 21, 2025

/terratest

@goruha goruha merged commit 2472cef into cloudposse:main Jan 21, 2025
26 checks passed
@mergify mergify bot removed the triage Needs triage label Jan 21, 2025
Copy link

These changes were released in v1.1.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants