How to delete disconnected sockets???

I use socket.io version 1.5.1 and I want to delete disconnected sockets, but I don’t find an answer to fix this.

This is the socket server in node

var http = require('http').Server(app);
var io = require('socket.io')(http);
var redis = require('redis').createClient();

redis.subscribe('change-financial-flag');
io.on('connection', function (socket){

    console.log("connected user: " + socket.id);
    clients.push(socket);

    redis.on('message', function(channel, message) {
        socket.emit(channel, message);
        console.log('emit: ' + channel + ' - socket: ' + socket.id);
    });

    socket.on('disconnect', function(){
        socket.disconnect(true);
        console.info('disconnected user (id=' + socket.id + ').');
    });
});

http.listen(5001, function () {
    console.log('listening on *:5001');
});

This is my output:

connected user: Bso5jOM3BmifSNUgAAAA
disconnected user (id=Bso5jOM3BmifSNUgAAAA).
connected user: yt0sLF5Mm5vts1ujAAAB
disconnected user (id=yt0sLF5Mm5vts1ujAAAB).
connected user: nZWM2gwE6pJXcIjsAAAC
disconnected user (id=nZWM2gwE6pJXcIjsAAAC).
connected user: MKhspxnSpmnxG4nAAAAD
disconnected user (id=MKhspxnSpmnxG4nAAAAD).
connected user: FnvUJsLKjtIC80MvAAAE
emit: change-financial-flag - socket: Bso5jOM3BmifSNUgAAAA
emit: change-financial-flag - socket: yt0sLF5Mm5vts1ujAAAB
emit: change-financial-flag - socket: nZWM2gwE6pJXcIjsAAAC
emit: change-financial-flag - socket: MKhspxnSpmnxG4nAAAAD
emit: change-financial-flag - socket: FnvUJsLKjtIC80MvAAAE

[marcelodiegues]

@darrachequesne Can you help me based in my code?

[darrachequesne]

The problem here is that you keep a reference to the socket in redis.on('message', ... ).

redis.subscribe('change-financial-flag');
redis.on('message', function(channel, message) {
    io.emit(channel, message); // emit to all connected users
    console.log('emit: ' + channel)
});

io.on('connection', function (socket){
    console.log("connected user: " + socket.id);

    socket.on('disconnect', function(){
        // socket.disconnect(true); <- that seems useless
        console.info('disconnected user (id=' + socket.id + ').');
    });
});