From 0bc6b967b4a8887f6035f9618b11cb1031beee55 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Alejandro=20Ferna=CC=81ndez=20Haro?=
 <alejandro.haro@elastic.co>
Date: Mon, 24 Aug 2020 09:49:37 +0100
Subject: [PATCH] Log the opt-in notification error

---
 src/plugins/telemetry/server/plugin.ts                |  1 +
 src/plugins/telemetry/server/routes/index.ts          |  3 ++-
 .../telemetry/server/routes/telemetry_opt_in.ts       | 11 ++++++++---
 3 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/src/plugins/telemetry/server/plugin.ts b/src/plugins/telemetry/server/plugin.ts
index 6c8888feafc1f..bd7a2a8c1a8ca 100644
--- a/src/plugins/telemetry/server/plugin.ts
+++ b/src/plugins/telemetry/server/plugin.ts
@@ -89,6 +89,7 @@ export class TelemetryPlugin implements Plugin {
       config$,
       currentKibanaVersion,
       isDev,
+      logger: this.logger,
       router,
       telemetryCollectionManager,
     });
diff --git a/src/plugins/telemetry/server/routes/index.ts b/src/plugins/telemetry/server/routes/index.ts
index ad84cb9d2665d..f46c616a734e0 100644
--- a/src/plugins/telemetry/server/routes/index.ts
+++ b/src/plugins/telemetry/server/routes/index.ts
@@ -18,7 +18,7 @@
  */
 
 import { Observable } from 'rxjs';
-import { IRouter } from 'kibana/server';
+import { IRouter, Logger } from 'kibana/server';
 import { TelemetryCollectionManagerPluginSetup } from 'src/plugins/telemetry_collection_manager/server';
 import { registerTelemetryOptInRoutes } from './telemetry_opt_in';
 import { registerTelemetryUsageStatsRoutes } from './telemetry_usage_stats';
@@ -28,6 +28,7 @@ import { TelemetryConfigType } from '../config';
 
 interface RegisterRoutesParams {
   isDev: boolean;
+  logger: Logger;
   config$: Observable<TelemetryConfigType>;
   currentKibanaVersion: string;
   router: IRouter;
diff --git a/src/plugins/telemetry/server/routes/telemetry_opt_in.ts b/src/plugins/telemetry/server/routes/telemetry_opt_in.ts
index 1d07856b4143c..aa1de4b2443a4 100644
--- a/src/plugins/telemetry/server/routes/telemetry_opt_in.ts
+++ b/src/plugins/telemetry/server/routes/telemetry_opt_in.ts
@@ -21,7 +21,7 @@ import moment from 'moment';
 import { Observable } from 'rxjs';
 import { take } from 'rxjs/operators';
 import { schema } from '@kbn/config-schema';
-import { IRouter } from 'kibana/server';
+import { IRouter, Logger } from 'kibana/server';
 import {
   StatsGetterConfig,
   TelemetryCollectionManagerPluginSetup,
@@ -39,12 +39,14 @@ import { TelemetryConfigType } from '../config';
 interface RegisterOptInRoutesParams {
   currentKibanaVersion: string;
   router: IRouter;
+  logger: Logger;
   config$: Observable<TelemetryConfigType>;
   telemetryCollectionManager: TelemetryCollectionManagerPluginSetup;
 }
 
 export function registerTelemetryOptInRoutes({
   config$,
+  logger,
   router,
   currentKibanaVersion,
   telemetryCollectionManager,
@@ -99,8 +101,11 @@ export function registerTelemetryOptInRoutes({
           telemetryCollectionManager,
           { optInStatusUrl, newOptInStatus },
           statsGetterConfig
-        ).catch(() => {
-          // Swallow this error. The server is likely behind a firewall and can't reach the remote service
+        ).catch((err) => {
+          // The server is likely behind a firewall and can't reach the remote service
+          logger.warn(
+            `Failed to notify "${optInStatusUrl}" from the server about the opt-in selection. Possibly blocked by a firewall? - Error: ${err.message}`
+          );
         });
       }