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

EREGCSC-update-allowed-hosts #993

Merged
merged 1 commit into from
Oct 13, 2023

Conversation

peggles2
Copy link
Contributor

@peggles2 peggles2 commented Oct 13, 2023

Resolves #EREGCSC-2257

Description-
We recently introduced a new domain, 'eregulations.cms.gov,' into our system and configured our API Gateway to accommodate it. However, we encountered an error in our AWS logs that reads: "ERROR Invalid HTTP_HOST header: 'eregulations.cms.gov.' The resolution to this issue involves adding 'eregulations.cms.gov' to the list of allowed hosts in our Django settings file, which, in our case, is 'base.py.' This adjustment will enable our application to properly handle requests from the 'eregulations.cms.gov' domain.

This pull request changes...

  • base.py

Steps to manually verify this change...
This may not be easy to verify if it fixes the issue till we push to prod. But we should check make sure it does not impact the current behavior. Recommend to visit experimental link, click around ensure the site is up and running.

All tests should pass.

@peggles2 peggles2 temporarily deployed to dev October 13, 2023 18:16 — with GitHub Actions Inactive
@peggles2 peggles2 temporarily deployed to dev October 13, 2023 18:31 — with GitHub Actions Inactive
Copy link
Contributor

@PhilR8 PhilR8 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@peggles2 peggles2 temporarily deployed to dev October 13, 2023 18:35 — with GitHub Actions Inactive
@peggles2 peggles2 temporarily deployed to dev October 13, 2023 18:35 — with GitHub Actions Inactive
@peggles2 peggles2 temporarily deployed to dev October 13, 2023 18:48 — with GitHub Actions Inactive
@github-actions
Copy link

✨ See the Django Site in action

@peggles2 peggles2 merged commit 06569c6 into main Oct 13, 2023
18 checks passed
@peggles2 peggles2 temporarily deployed to dev October 13, 2023 19:10 — with GitHub Actions Inactive
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