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

Handle Range requests explicitly #17

Merged
merged 1 commit into from Jan 5, 2015
Merged

Conversation

@kanongil
Copy link
Member

kanongil commented Dec 31, 2014

Fixes #6.

Note that all but the last test are copied from hapi. Additionally, I expect that this update will require the hapi tests to be reworked since the accept logic is no longer exercised for file responses.

Fixes #6
@kanongil kanongil added the feature label Dec 31, 2014
kanongil added a commit to kanongil/hapi that referenced this pull request Jan 4, 2015
This fixes missing test coverage with the new behavior in hapijs/inert#17
@hueniverse hueniverse added this to the 2.1.0 milestone Jan 5, 2015
@hueniverse hueniverse self-assigned this Jan 5, 2015
hueniverse added a commit that referenced this pull request Jan 5, 2015
Handle Range requests explicitly
@hueniverse hueniverse merged commit a6b3e67 into hapijs:master Jan 5, 2015
1 check passed
1 check passed
continuous-integration/travis-ci The Travis CI build passed
Details
@hueniverse

This comment has been minimized.

Copy link
Member

hueniverse commented Jan 5, 2015

Do we need to set the Vary header?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.