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.

Sign up for GitHub

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

Jump to bottom

Feat: Create middleware to bypass the ALLOWED_HOSTS validation #208

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Feat: Create middleware to bypass the ALLOWED_HOSTS validation #208

wants to merge 1 commit into from

Conversation

CaioTeixeira95
Copy link

Bypassing health_check endpoint.

Related to #45

@CaioTeixeira95 CaioTeixeira95 requested review from a team, matpfernandes, natamvo and vinibs and removed request for a team April 13, 2021 12:26
@CaioTeixeira95 CaioTeixeira95 changed the title Created middleware to bypass the ALLOWED_HOSTS validation WIP - Created middleware to bypass the ALLOWED_HOSTS validation Apr 13, 2021
@benmezger benmezger changed the title WIP - Created middleware to bypass the ALLOWED_HOSTS validation Created middleware to bypass the ALLOWED_HOSTS validation Apr 13, 2021
@benmezger benmezger marked this pull request as draft April 13, 2021 12:39
@benmezger
Copy link
Contributor

Moving this to a draft instead.

@benmezger benmezger requested a review from a team April 13, 2021 12:51
@CaioTeixeira95 CaioTeixeira95 changed the title Created middleware to bypass the ALLOWED_HOSTS validation Feat: Create middleware to bypass the ALLOWED_HOSTS validation Apr 15, 2021
@CaioTeixeira95 CaioTeixeira95 marked this pull request as ready for review April 15, 2021 14:33
@CaioTeixeira95 CaioTeixeira95 linked an issue Apr 15, 2021 that may be closed by this pull request
Create a health check endpoint that ignores ALLOWED_HOSTS check #45
Open
benmezger
benmezger approved these changes Apr 15, 2021
View reviewed changes
Copy link
Contributor

@benmezger benmezger left a comment

Choose a reason for hiding this comment

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

Thanks for the modifications!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@benmezger benmezger benmezger approved these changes

@matpfernandes matpfernandes Awaiting requested review from matpfernandes

@natamvo natamvo Awaiting requested review from natamvo

At least 5 approving reviews are required to merge this pull request.

Assignees

@CaioTeixeira95 CaioTeixeira95

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create a health check endpoint that ignores ALLOWED_HOSTS check
2 participants
@CaioTeixeira95 @benmezger