Skip to content

Support HTTP/1.1 and HTTP/1.0 - #125

Merged
WyriHaximus merged 4 commits into
reactphp:masterfrom
clue-labs:protocol-version
Feb 19, 2017
Merged

Support HTTP/1.1 and HTTP/1.0#125
WyriHaximus merged 4 commits into
reactphp:masterfrom
clue-labs:protocol-version

Conversation

@clue

@clueclue commented Feb 19, 2017

Copy link
Copy Markdown
Member
  • Response uses same HTTP protocol version as corresponding request
  • Apply chunked transfer encoding only for HTTP/1.1 responses by default
  • Ensure writeContinue() only works for HTTP/1.1 messages
  • Only support HTTP/1.1 and HTTP/1.0 requests

Closes#121

@clueclue added this to the v0.6.0 milestone Feb 19, 2017
WyriHaximus
WyriHaximus approved these changes Feb 19, 2017
View reviewed changes

@WyriHaximusWyriHaximus left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

@WyriHaximus
WyriHaximus merged commit 61a69d7 into reactphp:masterFeb 19, 2017
This was referenced Feb 19, 2017
Merged
Merged
@clue
clue deleted the protocol-version branch February 19, 2017 15:12
@clueclue mentioned this pull request Feb 19, 2017
Merged
@clueclue mentioned this pull request Feb 19, 2017
Merged
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jsorjsorjsor approved these changes
@WyriHaximusWyriHaximusWyriHaximus approved these changes

Assignees

No one assigned

Labels

HTTP/1.0HTTP/1.1new feature

Projects

None yet

Milestone

v0.6.0

Development

Successfully merging this pull request may close these issues.

3 participants

@clue@jsor@WyriHaximus