memory leak fix (by argon) and another 'upstream is not defined' check

Open simllll opened this issue 9 years ago • 4 comments

this PR contains Clean up ended streams to free leaked memory #200 and an additional check if stream is defined. (line 314)

without these changes I was not able to run this code for a longer time. I use this in production, should be well "tested".

Oct 20 '16 09:10 simllll

@simllll Are you still using this in production? Looking at just hosting a zip and referencing from my own package. Can't believe no movement here...

Jan 10 '17 17:01 AndrewBarba

Hey Andrew,yes still using it. True that,not sure what happened to the repo owner?!

Jan 11 '17 07:01 simllll

hope merge as soon as possible

Feb 13 '17 02:02 haozxuan

was merged via https://github.com/kaazing/node-http2/commit/db0936244be639cbfd9a45c85377d0923a123b46

Aug 23 '17 20:08 hthetiot