Skip to content
This repository has been archived by the owner on Dec 18, 2018. It is now read-only.

Reject requests that don't have a Host header #839

Closed
cesarblum opened this issue May 18, 2016 · 2 comments
Closed

Reject requests that don't have a Host header #839

cesarblum opened this issue May 18, 2016 · 2 comments
Assignees
@natemcmaster
Labels
3 - Done enhancement
Milestone
2.0.0-preview1

Comments

@cesarblum
Copy link
Contributor

cesarblum commented May 18, 2016 •
edited by natemcmaster
Loading

RFC 7230 section 5.4 - https://tools.ietf.org/html/rfc7230#section-5.4

A server MUST respond with a 400 (Bad Request) status code to any HTTP/1.1 request message that lacks a Host header field and to any request message that contains more than one Host header field or a Host header field with an invalid field-value.
@cesarblum cesarblum changed the title Reject request that don't have a Host header Reject requests that don't have a Host header May 18, 2016
@muratg
Copy link
Contributor

muratg commented May 18, 2016

@CesarBS What's our current behavior for requests that don't have a Host header?

@cesarblum
Copy link
Contributor Author

We process them as we would any other request.

@muratg muratg added this to the Backlog milestone May 18, 2016
@muratg muratg removed this from the Backlog milestone Oct 12, 2016
@muratg muratg added this to the 1.2.0 milestone Oct 31, 2016
@khellang khellang mentioned this issue Nov 22, 2016
Closed
@muratg muratg modified the milestones: 1.2.0, 2.0.0 Jan 12, 2017
@muratg muratg assigned muratg and natemcmaster and unassigned muratg Apr 7, 2017
@shirhatti shirhatti mentioned this issue Apr 8, 2017
Closed
@natemcmaster natemcmaster mentioned this issue Apr 12, 2017
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@natemcmaster natemcmaster

Labels
3 - Done enhancement
Projects
None yet
Milestone
2.0.0-preview1
Development

No branches or pull requests
3 participants
@cesarblum @muratg @natemcmaster