Unify socket closers. Don't put them all in one try-catch or some might remain.

lib/units/notify/hipchat.js

@@ -72,6 +72,11 @@ module.exports = function(options) {
     logger.LevelLabel[options.priority])
 
   lifecycle.observe(function() {
-    sub.close()
+    try {
+      sub.close()
+    }
+    catch (err) {
+      // No-op
+    }
   })
 }

lib/units/processor/index.js

@@ -161,10 +161,13 @@ module.exports = function(options) {
     .handler())
 
   lifecycle.observe(function() {
-    try {
-      appDealer.close()
-      devDealer.close()
-    }
-    catch (err) {}
+    [appDealer, devDealer].forEach(function(sock) {
+      try {
+        sock.close()
+      }
+      catch (err) {
+        // No-op
+      }
+    })
   })
 }

lib/units/provider/index.js

@@ -394,13 +394,16 @@ module.exports = function(options) {
   })()
 
   lifecycle.observe(function() {
-    clearTimeout(totalsTimer)
+    [push, sub].forEach(function(sock) {
+      try {
+        sock.close()
+      }
+      catch (err) {
+        // No-op
+      }
+    })
 
-    try {
-      push.close()
-      sub.close()
-    }
-    catch (err) {}
+    clearTimeout(totalsTimer)
 
     return Promise.all(Object.keys(workers).map(function(serial) {
       return workers[serial].cancel()

lib/units/reaper/index.js

@@ -53,6 +53,8 @@ module.exports = function(options) {
     try {
       push.close()
     }
-    catch (err) {}
+    catch (err) {
+      // No-op
+    }
   })
 }

lib/units/triproxy/index.js

@@ -34,11 +34,13 @@ module.exports = function(options) {
   log.info('PULL socket bound on', options.endpoints.pull)
 
   lifecycle.observe(function() {
-    try {
-      pub.close()
-      dealer.close()
-      pull.close()
-    }
-    catch (err) {}
+    [pub, dealer, pull].forEach(function(sock) {
+      try {
+        sock.close()
+      }
+      catch (err) {
+        // No-op
+      }
+    })
   })
 }