From 7ac62da3c90b4d00d5852bb9488af509ed01b4fb Mon Sep 17 00:00:00 2001 From: Cryptkeeper Date: Tue, 24 Nov 2015 00:34:00 -0600 Subject: [PATCH] Fix: Delete old favicons, save us tons of bandwidth --- app.js | 26 ++++++++++++++++++-------- 1 file changed, 18 insertions(+), 8 deletions(-) diff --git a/app.js b/app.js index 654eb73..9c1ddc3 100644 --- a/app.js +++ b/app.js @@ -26,14 +26,20 @@ function pingAll() { res.favicon = config.faviconOverride[network.name]; } - server.io.sockets.emit('update', { - result: res, - error: err, - info: { - name: network.name, - timestamp: util.getCurrentTimeMs() - } - }); + var networkSnapshot = { + info: { + name: network.name, + timestamp: util.getCurrentTimeMs() + } + }; + + if (networkSnapshot) { + object.result = res; + } else if (err) { + networkSnapshot.err = error; + } + + server.io.sockets.emit('update', networkSnapshot); // Log our response. if (!networkHistory[network.ip]) { @@ -45,6 +51,10 @@ function pingAll() { // Remove our previous data that we don't need anymore. for (var i = 0; i < _networkHistory.length; i++) { delete _networkHistory[i].info; + + if (_networkHistory[i].result) { + delete _networkHistory[i].result.favicon; + } } _networkHistory.push({