Stopped logging after loosing connectivity to papertrail about pino-papertrail

[mattiash]

The "problem" is that pumpify uses pump, and the documentation for pump explicitly says:

Combine an array of streams into a single duplex stream using pump and duplexify. If one of the streams closes/errors all streams in the pipeline will be destroyed.

I looked at the code in this module and came up with the following patch:

diff --git a/lib/pino-papertrail.js b/lib/pino-papertrail.js
index 0931f6a..efaedbf 100644
--- a/lib/pino-papertrail.js
+++ b/lib/pino-papertrail.js
@@ -86,7 +86,10 @@ function toPapertrailUdp (options) {
     write (data, encoding, callback) {
       if (options.echo === true) { console.log(data.toString()) }
       socket.send(data, 0, data.length, options.port, options.host, function (err) {
-        callback(err)
+        if(err) {
+          console.log('error', err.message)
+        }
+        callback()
       })
     }
   })

What it does is to not pass the err value to the callback, i.e. just swallow all the data and don't report an error to the callback even if it fails to send the data to papertrail. I think this is an acceptable behavior, since we are using udp anyway and that does not give any guarantees about delivery.

With this patch, pino-papertrail will log an error while DNS lookups fail and then resume sending data to papertrail when DNS lookups work again.

My question is if this is an acceptable approach to you and what I should do with the error. Just logging it to stdout seems reasonable to me.

[ovhemert]

LGTM,.. feel free to send a PR