From 963b90edf04e23d7cb8abf3c84da33cf07a4dbc8 Mon Sep 17 00:00:00 2001 From: Devon Carew Date: Wed, 21 Jun 2017 13:02:40 -0700 Subject: [PATCH] send the FLUTTER_HOST env var as analytics' app installer ID (#10889) --- packages/flutter_tools/lib/src/usage.dart | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/packages/flutter_tools/lib/src/usage.dart b/packages/flutter_tools/lib/src/usage.dart index 7953eba0b9c..a5d9683958d 100644 --- a/packages/flutter_tools/lib/src/usage.dart +++ b/packages/flutter_tools/lib/src/usage.dart @@ -9,12 +9,11 @@ import 'package:usage/usage_io.dart'; import 'base/context.dart'; import 'base/os.dart'; +import 'base/platform.dart'; import 'base/utils.dart'; import 'globals.dart'; import 'version.dart'; -// TODO(devoncarew): We'll want to find a way to send (sanitized) command parameters. - const String _kFlutterUA = 'UA-67589403-6'; Usage get flutterUsage => Usage.instance; @@ -25,10 +24,14 @@ class Usage { final String version = versionOverride ?? FlutterVersion.getVersionString(whitelistBranchName: true); _analytics = new AnalyticsIO(_kFlutterUA, settingsName, version); - // Report a more detailed OS version string than package:usage does by - // default. Also, send the branch name as the "channel". + // Report a more detailed OS version string than package:usage does by default. _analytics.setSessionValue('cd1', os.name); + // Send the branch name as the "channel". _analytics.setSessionValue('cd2', FlutterVersion.getBranchName(whitelistBranchName: true)); + // Record the host as the application installer ID - the context that flutter_tools is running in. + if (platform.environment.containsKey('FLUTTER_HOST')) { + _analytics.setSessionValue('aiid', platform.environment['FLUTTER_HOST']); + } bool runningOnCI = false;