d5 / node.native

C++11 port for the Node: native performance and modern simplicity.
MIT License
1.16k stars 149 forks source link

Keep-Alive support #31

Open sebjameswml opened 10 years ago

sebjameswml commented 10 years ago

I want to implement Keep-Alive support in node.native.

I can easily enough set a response to tell the client that the connection should be kept alive.

What I'm having a problem with is executing a callback when a second request comes from the client on the kept-alive connection.

In node.js and luvit, they have an on:data callback. I can't find this event in libuv, so I'm really asking you all if you know if node.js/luvit implement additional events beyond those which libuv provides.

sebjameswml commented 10 years ago

native/events.h is the definition of various callbacks in node.native. So I need to add an dev:ev::data callback to the http classes to make my Keep-Alive support work.

sebjameswml commented 10 years ago

See sebjameswml/node.native for the keepalive support; commit c5c93980b8df699804f88db34e9f82566db7bf7c

This adds keepaliveserver.cpp and keepaliveclient.cpp, along with changes in http.h.

sebjameswml commented 10 years ago

@dennycd Do you want me to pull-request these changes?