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

fix: Switched back to using ELB instead of NLB due to various issues. Also added the ability to modify proxy protocol and traffic policy settings and return the nginx request id to the front end. #45

Merged
merged 2 commits into from
Apr 23, 2021

Conversation

bmonkman
Copy link
Contributor

Description

Please explain the changes you made here and link to any relevant issues.

Checklist

… Also added the ability to modify proxy protocol and traffic policy settings and return the nginx request id to the front end.
@bmonkman bmonkman requested a review from a team as a code owner April 23, 2021 20:45
@bmonkman bmonkman requested a review from davidcheung April 23, 2021 20:45
@github-actions
Copy link

Terraform Format and Style 🖌success

Terraform Initialization ⚙️success

Terraform Validation 🤖Success! The configuration is valid.

@github-actions
Copy link

Terraform Format and Style 🖌success

Terraform Initialization ⚙️success

Terraform Validation 🤖Success! The configuration is valid.

@bmonkman bmonkman merged commit 4851b28 into main Apr 23, 2021
@bmonkman bmonkman deleted the switch-back-to-elb-by-default branch April 23, 2021 22:24
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.

2 participants