Use finished instead of on-error

lib/monitor.js

@@ -1,7 +1,7 @@
 'use strict';
 
 const Os = require('os');
-const { pipeline } = require('stream');
+const { pipeline, finished } = require('stream');
 
 const Hoek = require('@hapi/hoek');
 const Oppsy = require('@hapi/oppsy');
@@ -121,8 +121,10 @@ module.exports = internals.Monitor = class {
 
             this._reporters.set(reporterName, setupPipeline(streamObjs, (err) => {
 
-                console.error(`There was a problem (${err}) in ${reporterName} and it has been destroyed.`);
-                console.error(err);
+                if (err) {
+                    console.error(`There was a problem (${err}) in ${reporterName} and it has been destroyed.`);
+                    console.error(err);
+                }
             }));
         }
 
@@ -205,20 +207,15 @@ module.exports = internals.Monitor = class {
     }
 };
 
-const setupPipeline = (streams, onError) => {
+const setupPipeline = (streams, onFinished) => {
 
     const stream = streams[0];
 
     if (streams.length === 1) {
-        stream.on('error', onError);
+        finished(stream, onFinished);
     }
     else {
-        pipeline(streams, (err) => {
-
-            if (err) {
-                onError(err);
-            }
-        });
+        pipeline(streams, onFinished);
     }
 
     return stream;