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

Wrap recvN in HTTP2 to tickle the timer. #624

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@lucasdicioccio
Contributor

lucasdicioccio commented May 30, 2017

This patch addresses a bug preventing long HTTP2 connections. See #593
for an initial report, as well as
#593 (comment) for a
reproducible example -- which this patch addresses (the actual root
cause for #593 might differ but seem closely related).

Wrap recvN in HTTP2 to tickle the timer.
This patch addresses a bug preventing long HTTP2 connections.  See #593
for an initial report, as well as
#593 (comment) for a
reproducible example -- which this patch addresses (the actual root
cause for #593 might differ but seem closely related).

kazu-yamamoto added a commit to kazu-yamamoto/wai that referenced this pull request Jul 3, 2017

@kazu-yamamoto

This comment has been minimized.

Contributor

kazu-yamamoto commented Jul 3, 2017

I have merged this PR with rebasing and modifications.
Thank you for your contribution.

@kazu-yamamoto

This comment has been minimized.

Contributor

kazu-yamamoto commented Jul 4, 2017

A new version has been released.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment