Fixed API crashing if a batch miner stats lookup isn't successful

SirEOF · Jun 22, 2014 · c641e86 · c641e86
1 parent 0551ad0
commit c641e86
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/api.js b/lib/api.js
@@ -186,7 +186,7 @@ function broadcastLiveStats(){
             var address = addresses[i];
             var stats = replies[i];
             var res = addressConnections[address];
-            res.end(formatMinerStats(stats, address));
+            res.end(stats ? formatMinerStats(stats, address) : '{"error": "not found"');
         }
     });
 }

diff --git a/lib/pool.js b/lib/pool.js
@@ -381,7 +381,7 @@ function recordShareData(miner, job, shareDiff, blockCandidate, hashHex, shareTy
         }
     });
 
-    log('debug', 'Share Validator', 'Accepted ' + shareType + 'share at difficulty ' + job.difficulty + '/' + shareDiff + ' from ' + miner.login + '@' + miner.ip);
+    log('debug', 'Share Validator', 'Accepted ' + shareType + ' share at difficulty ' + job.difficulty + '/' + shareDiff + ' from ' + miner.login + '@' + miner.ip);
 
 }
-Original file line number
+Diff line change
@@ Expand Up @@
             }
         });
-        log('debug', 'Share Validator', 'Accepted ' + shareType + 'share at difficulty ' + job.difficulty + '/' + shareDiff + ' from ' + miner.login + '@' + miner.ip);
+        log('debug', 'Share Validator', 'Accepted ' + shareType + ' share at difficulty ' + job.difficulty + '/' + shareDiff + ' from ' + miner.login + '@' + miner.ip);
     }
@@ Expand Down @@