Return Boom errors directly to the browser for Time Series Visual Bui…

…lder (elastic#11656)

* Fixed elastic#11643 - Return Boom errors directly to the browser

* Checking for 401 and boom errors instead of just boom errors

* removing the returns from the hapi routes

src/core_plugins/metrics/server/lib/vis_data/handle_error_response.js

@@ -1,5 +1,5 @@
 export default panel => error => {
-  console.log(error);
+  if (error.isBoom && error.status === 401) throw error;
   const result = {};
   let errorResponse;
   try {

src/core_plugins/metrics/server/routes/fields.js

@@ -5,9 +5,12 @@ export default (server) => {
     path: '/api/metrics/fields',
     method: 'GET',
     handler: (req, reply) => {
-      return getFields(req)
+      getFields(req)
         .then(reply)
-        .catch(() => reply([]));
+        .catch((err) => {
+          if (err.isBoom && err.status === 401) return reply(err);
+          reply([]);
+        });
     }
   });
 

src/core_plugins/metrics/server/routes/vis.js

@@ -6,10 +6,9 @@ export default (server) => {
     path: '/api/metrics/vis/data',
     method: 'POST',
     handler: (req, reply) => {
-      return getVisData(req)
+      getVisData(req)
         .then(reply)
         .catch(err => {
-          console.error(err.stack);
           reply(Boom.wrap(err, 400));
         });
     }