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.

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

Keep-Alive support #31

Open
sebjameswml opened this issue Nov 25, 2013 · 3 comments
Open

Keep-Alive support #31

sebjameswml opened this issue Nov 25, 2013 · 3 comments

Comments

@sebjameswml
Copy link
Contributor

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
Copy link
Contributor Author

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
Copy link
Contributor Author

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

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

@sebjameswml
Copy link
Contributor Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant