Skip to content

Do not pause connection stream to detect closed connections immediately - #315

Merged
jsor merged 1 commit into
reactphp:masterfrom
clue-labs:close-immediately
Apr 9, 2018
Merged

Do not pause connection stream to detect closed connections immediately#315
jsor merged 1 commit into
reactphp:masterfrom
clue-labs:close-immediately

Conversation

@clue

@clueclue commented Apr 6, 2018

Copy link
Copy Markdown
Member

Resolves / closes#210
Builds on top of #188
Prerequisite for #39

@clueclue added this to the v0.8.3 milestone Apr 6, 2018
@clue
clueforce-pushed the close-immediately branch from 0e88ff0 to 65dc863CompareApril 6, 2018 11:56
@clue
clue requested review from WyriHaximus and jsorApril 6, 2018 12:04
@clue
clueforce-pushed the close-immediately branch from 65dc863 to 3e45972CompareApril 7, 2018 13:59
@clue

clue commented Apr 7, 2018

Copy link
Copy Markdown
MemberAuthor

Rebased to resolve merge conflict now that #317 is in :shipit:

jsor
jsor approved these changes Apr 9, 2018
@jsor
jsor merged commit 6afc50b into reactphp:masterApr 9, 2018
@clue
clue deleted the close-immediately branch April 9, 2018 06:55
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HttpBodyStream not fired close immediate when browser is close

3 participants

@clue@jsor@WyriHaximus