Catch more stasd exceptions

Closes #167
CartoDB · Mar 4, 2014 · fe41b04 · fe41b04
1 parent 18f8642
commit fe41b04
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 4 deletions.
diff --git a/NEWS.md b/NEWS.md
@@ -3,7 +3,7 @@ Version 0.19.4
 
  Enhancements:
 
- - Catch profiler exceptions (#166)
+ - Catch statsd exceptions (#166, #167)
 
 Version 0.19.3
 2014-03-04

diff --git a/lib/windshaft/server.js b/lib/windshaft/server.js
@@ -786,8 +786,12 @@ module.exports = function(opts){
             if ( err ) throw err;
             app.getTileOrGrid(req, res, this); // legacy get map tile endpoint
           },
-          function finish(err, req_ret, res_ret, tile, headers) {
+          function finalize(err, req_ret, res_ret, tile, headers) {
             app.finalizeGetTileOrGrid(err, req, res, tile, headers);
+            return null;
+          },
+          function finish(err) {
+            if ( err ) console.error("windshaft.tiles: " + err);
           }
         );
     });
@@ -1192,8 +1196,12 @@ module.exports = function(opts){
           if ( err ) throw err;
           app.getTileOrGrid(req, res, this); // map api map tile endpoint
         },
-        function finish(err, req_ret, res_ret, tile, headers) {
+        function finalize(err, req_ret, res_ret, tile, headers) {
           app.finalizeGetTileOrGrid(err, req, res, tile, headers);
+          return null;
+        },
+        function finish(err) {
+          if ( err ) console.error("windshaft.tiles: " + err);
         }
       );
     });
@@ -1216,8 +1224,12 @@ module.exports = function(opts){
           if ( err ) throw err;
           app.getTileOrGrid(req, res, this); // map api map layer tile endpoint
         },
-        function finish(err, req_ret, res_ret, tile, headers) {
+        function finalize(err, req_ret, res_ret, tile, headers) {
           app.finalizeGetTileOrGrid(err, req, res, tile, headers);
+          return null;
+        },
+        function finish(err) {
+          if ( err ) console.error("windshaft.tiles: " + err);
         }
       );
     });