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

[CORL-867] Health SSL Bypass #2796

Merged
merged 2 commits into from
Jan 15, 2020
Merged

[CORL-867] Health SSL Bypass #2796

merged 2 commits into from
Jan 15, 2020

Conversation

wyattjoh
Copy link
Collaborator

What does this PR do?

Bypasses enforcing SSL requirement for the /api/health endpoint as this is typically accessed internally by the infrastructure.

How do I test this PR?

curl -v http://localhost:3000/api/health

Notice that it returns 200, not a 301.

@wyattjoh wyattjoh added the bug label Jan 15, 2020
@wyattjoh wyattjoh added this to the v5.4.1 milestone Jan 15, 2020
@wyattjoh wyattjoh requested a review from tessalt January 15, 2020 19:24
@wyattjoh wyattjoh merged commit f58f65c into master Jan 15, 2020
@wyattjoh wyattjoh deleted the bugfix/CORL-867 branch January 15, 2020 20:11
@wyattjoh wyattjoh mentioned this pull request Jan 15, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tessalt tessalt tessalt approved these changes

Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
v5.4.1
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@wyattjoh @tessalt