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.

Sign up for GitHub

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

Fixing things I screwed up with evhtp. #1462

Closed
wants to merge 1 commit into from
Closed

Fixing things I screwed up with evhtp. #1462

wants to merge 1 commit into from

Commits on Nov 29, 2015

  1. Fixing things I screwed up with evhtp. 

    - By screwing up, I mean, a total lack of communication about this change. And I
  wasn't aware of th problem until just now. But that's no excuse.

  The semi-explaination I posted here: explaination can be found here:
  #1119

  Which links to another 500 page issue that caused the logic to change, ever so
  slightly, an issue that sneaks up on you if unaware (thus my apologies for the
  lack of communication)

  Kinda like the day I broke tor.

  Gist is, you can get into a state of hung connections if you send data right ifter a
  resume. You have to treat it like pending data, then emit it to get the event
  loop rolling again.

  I'm pretty sure this should fix the problem, if not, we have some more safety
  nets in our develop tree which may further the bugginess of resumption +
  pipelining.
    Mark Ellzey committed Nov 29, 2015
    Configuration menu
    Copy the full SHA
    6cb33ce View commit details
    Browse the repository at this point in the history