[Telemetry] Error handling: local stats collection/permissions bug fix (#26560) (#26567)

* monitoring collection bug fix

* show error for dev
This commit is contained in:
Tim Sullivan 2018-12-03 17:03:18 -07:00 committed by GitHub
parent 8f48e81ec7
commit 7ca581c4a9
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -21,7 +21,12 @@ export async function getTelemetry(req, config, start, end, { _getAllStats = get
let response = [];
if (config.get('xpack.monitoring.enabled')) {
response = await _getAllStats(req, start, end);
try {
// attempt to collect stats from multiple clusters in monitoring data
response = await _getAllStats(req, start, end);
} catch (err) {
// no-op
}
}
if (!Array.isArray(response) || response.length === 0) {
@ -92,7 +97,7 @@ export function telemetryRoute(server) {
} catch (err) {
if (config.get('env.dev')) {
// don't ignore errors when running in dev mode
return boomify(err);
return boomify(err, { statusCode: err.status });
} else {
// ignore errors, return empty set and a 200
return h.response([]).code(200);