spdy-push icon indicating copy to clipboard operation
spdy-push copied to clipboard

Published 20 hours ago • verified publisher icon jshttp

Update spdy dep

Open rektide opened this issue 10 years ago • 5 comments

Would be great to get spdy v2 here; latest is 2.0.5. Currently at "1".

rektide avatar Sep 26 '15 19:09 rektide

trying to upgrade, but getting a GOAWAY error in the tests. any ideas?

jonathanong avatar Sep 27 '15 02:09 jonathanong

There's a bunch of api changes, it seems there's a bunch of places where callbacks must now be provided in node-spdy. I've just been monkey-see/monkey-do'ing, & have a WIP branch. There's now a proper exchange of messages again, a message in and a push. It's not refined, nor finished, but at least is on the right track to making everything go again in node-spdy2.

However the acknowledge event is no longer emitted, and I'm not sure what the replacement is going to be. The afterEach is also not properly terminating; apparently the server is not closing down?

rektide avatar Sep 29 '15 07:09 rektide

thank you! :D i haven't had time to look at it

jonathanong avatar Sep 29 '15 21:09 jonathanong

@rektide any idea what the request and response options are in .push()?

jonathanong avatar Sep 30 '15 02:09 jonathanong

dealing with some stupid GOAWAY error...

jonathanong avatar Feb 16 '16 06:02 jonathanong