Change optIn to default to true

elastic · Nov 13, 2019 · 46f4ba2 · 46f4ba2
1 parent 7bb968c
commit 46f4ba2
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 6 deletions.
diff --git a/src/legacy/core_plugins/telemetry/index.ts b/src/legacy/core_plugins/telemetry/index.ts
@@ -50,10 +50,8 @@ const telemetry = (kibana: any) => {
         allowChangingOptInStatus: Joi.boolean().default(true),
         optIn: Joi.when('allowChangingOptInStatus', {
           is: false,
-          then: Joi.valid(true).required(),
-          otherwise: Joi.boolean()
-            .allow(null)
-            .default(null),
+          then: Joi.valid(true).default(true),
+          otherwise: Joi.boolean().default(true),
         }),
 
         // `config` is used internally and not intended to be set

diff --git a/src/legacy/core_plugins/telemetry/public/components/telemetry_form.js b/src/legacy/core_plugins/telemetry/public/components/telemetry_form.js
@@ -116,7 +116,7 @@ export class TelemetryForm extends Component {
                 type: 'boolean',
                 value: telemetryOptInProvider.getOptIn() || false,
                 description: this.renderDescription(),
-                defVal: false,
+                defVal: true,
               }}
               save={this.toggleOptIn}
               clear={this.toggleOptIn}

diff --git a/src/legacy/core_plugins/telemetry/server/telemetry_config/get_telemetry_opt_in.ts b/src/legacy/core_plugins/telemetry/server/telemetry_config/get_telemetry_opt_in.ts
@@ -44,7 +44,7 @@ export const getTelemetryOptIn: GetTelemetryOptIn = ({
   }
 
   if (telemetrySavedObject === null || typeof telemetrySavedObject.enabled !== 'boolean') {
-    return null;
+    return configTelemetryOptIn;
   }
 
   const savedOptIn = telemetrySavedObject.enabled;