nodejs, socket.io einfacher Code Speicherleck

Question

Ich schrieb Code unter Verwendung von Nodejs und socket.io für eine einfache Socket-Anwendung (nur verbinden und trennen), für etwa 50 Benutzer Speicherverbrauch nicht zu viel ändern, aber für bis zu 300 Benutzer und nach einer Stunde, Speicherverbrauch nur wachsen (in der Nähe von 300 MB für server.js Prozess und wächst nach Zeit verstreichen), es sieht aus wie diese Nodejs nicht Speicher freigeben.

var server = require('http').createServer(); 
var io = require('socket.io')(server); 
var port = 9090; 
var sockets = {}; 

server.listen(port, function() { 
    console.log('Server listening at port ', port); 
    //timer and logs are not problem , i tested it before. 
    setInterval(function(){ 
     console.log(Object.keys(sockets).length+' Online Devices At '+Date()); 
    }, 1000 * 60 * 1); 
}); 

io.on('connection',function(socket){ 
    sockets[socket.id]={id:socket.id}; 
    console.log('connected '+socket.id + ' count ' + Object.keys(sockets).length); 
    socket.on('disconnect', function (data) { 
     delete sockets[socket.id]; 
     console.log('disconnected '+socket.id+ ' count ' +Object.keys(sockets).length); 
    }); 
}); 

mache ich etwas falsch ?!

bearbeiten

14 Stunden nach forever

300 offenen Steckdosen mit Datei starten und über 500 MB Speichernutzung, die zu meinem NodeJS proccess verwendet ist.

bearbeiten

Nach 16 Stunden, 300 Connected Sockets Nach Prozess gestoppt.

bearbeiten

Beute an meinem neuen Code Bitte übertragen.

var server = require('http').createServer(); 
var io = require('socket.io')(server); 
var port = 90; 
var counter = 0; 
var clients = {} 
server.listen(port, function() { 
     console.log('Server listening at port ', port); 
}); 
io.on("connection",function(socket){ 
     clients[socket.id] = socket; 
     counter++; 

     socket.on('disconnect', function (data) { 
       counter--; 
       delete clients[socket.id]; 
     }); 
}); 

Ich versuche, dies mit 1000 angeschlossenen Benutzern (ein anderer Server ist Simulieren Benutzeranforderungen und offene Steckdosen)

Speichernutzung vor dem Start: 100MB, nach 5 Minuten und 1000 stabil offenen Verbindungen: 400MB

Answer 1

Der Code sieht gut aus. Ihr vorgeschlagener Speicherverlust ist fast sicher nicht in dem Teil des Codes, den Sie freigegeben haben.

Dies ist zu Ihrer Hauptfrage nicht relevant, aber wenn Sie nur die Anzahl der angeschlossenen Steckdosen zur Liste möchten, sollten Sie einen Integer-Zähler anstelle Object.keys() der Aufruf auf dem sockets Objekt, wie folgt aus:

var express = require('express'); 
var app  = express(); 
var server = require('http').createServer(app); 
var io  = require('socket.io')(server); 

var port = 9090; 
var connectedSockets = 0; 
var sockets = {}; 

server.listen(port, function() { 
    console.log('Server listening at port ', port); 
    //timer and logs are not problem , i tested it before. 
    setInterval(function(){ 
     console.log(connectedSockets + ' Online Devices At ' + Date()); 
    }, 1000 * 60 * 1); 
}); 

io.on('connection',function(socket){ 
    if (!sockets[socket.id]) connectedSockets++; 
    sockets[socket.id]={ id: socket.id }; 
    console.log('connected ' + socket.id + ' count ' + connectedSockets); 
    socket.on('disconnect', function (data) { 
     delete sockets[socket.id]; 
     connectedSockets--; 
     console.log('disconnected ' + socket.id + ' count ' + connectedSockets); 
    }); 
}); 

Answer 2

V8 ist faul, wenn es darum geht, ungenutzten Speicher freizugeben, so dass es wie ein Speicherleck aussieht, wenn V8 gerade nicht seinen Garbage Collector ausführt. Um zu sehen, ob dies der Fall ist, führen Sie Ihren Prozess mit dem --expose-gc Flag aus, das z.

node --expose-gc yourscript.js 

Und erzwingen manuelle Garbage Collection in einem Intervall (Ich habe 30 Sekunden Intervall).

setInterval(function(){ 
    global.gc(); 
    console.log('GC done') 
}, 1000*30);