search

daguej / node-proxywrap

Wrap node's Server interfaces to be compatible with the PROXY protocol
48 stars 36 forks source link

Sockets are leaking #9

Open jaslo opened 10 years ago

jaslo commented 10 years ago

Sockets are leaking when I set up servers using proxywrap -- eventually my server fails with messages of "Error: accept EMFILE". And I could see the growth of the open sockets in /proc/id/fd

daguej commented 10 years ago

I will look in to this, but probably won't have time within the next couple of weeks. In the meantime, any additional information/test cases/debug dumps would be useful.

What version of node and Linux?

jaslo commented 10 years ago

node 10.29 ubuntu 13.10

When I remove the proxywrap and just use an http server, the leaks go away!

On Mon, Aug 18, 2014 at 5:04 PM, Josh Dague notifications@github.com wrote:

I will look in to this, but probably won't have time within the next couple of weeks. In the meantime, any additional information/test cases/debug dumps would be useful.

What version of node and Linux?

— Reply to this email directly or view it on GitHub https://github.com/daguej/node-proxywrap/issues/9#issuecomment-52573787.

cusspvz commented 10 years ago

@daguej, i've created a fork of node-proxywrap here, we have to maintain the repos we use if they are without recent activity.

@jaslo could you provide a test unit on findhit-proxywrap so we can solve this issue?