setting a timeout for every new socket node.js

Question

As I'm building my own http module relying on net module, I want to set a timeout for every new socket. I build the following:

server = net.createServer(function (socket) {
          if(server.timeout==null){
                server.timeout=120000;
                if(server.timeoutCallBack==null){
                    server.timeoutCallBack=function(){socket.destroy();};
                }
            }
          socket.setTimeout(server.timeout,server.timeoutCallBack());

so if the user of my server, set a new timeout duration , it will use it. if not, it has a default value of 2 minutes.

My problem is that my server doesn't serve the first connection, just the second. I mean, for the first http message it gets, it doesn't invoke my :

            socket.on("data", function (data) {
            });

Tibos · Accepted Answer · 2014-01-11 08:48:38Z

4

The thing you passed to socket.setTimeout as your callback is the result of server.timeoutCallBack(), which is undefined. Instead of calling timeoutCallBack 2 minutes after that line, you called it on that line and setTimeout of nothing.

You should pass a reference to a function to setTimeout instead:

socket.setTimeout(server.timeout,server.timeoutCallBack);

answered Jan 11, 2014 at 8:48

Tibos

27.9k4 gold badges56 silver badges66 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

setting a timeout for every new socket node.js

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related