feat(GUI): use resin-raven in AnalyticsService

This commit also removes bower. Change-Type: patch
balena-io · Mar 24, 2017 · 17592de · 17592de
1 parent 063fde1
commit 17592de
Show file tree

Hide file tree

Showing 15 changed files with 857 additions and 2,351 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -37,7 +37,7 @@ install:
       ./scripts/build/docker/run-command.sh -r ${TARGET_ARCH} -s ${PWD} -c "make info && make electron-develop";
     else
       pip install codespell==1.9.2 awscli;
-      npm install -g bower asar;
+      npm install -g asar;
       brew install afsctool jq;
       make info;
       make electron-develop;

diff --git a/Makefile b/Makefile
@@ -189,18 +189,13 @@ $(BUILD_DIRECTORY)/node-$(TARGET_PLATFORM)-$(TARGET_ARCH)-dependencies/node_modu
 		-t node \
 		-s "$(TARGET_PLATFORM)"
 
-$(BUILD_DIRECTORY)/electron-$(TARGET_PLATFORM)-$(TARGET_ARCH)-dependencies/bower_components: bower.json \
-	| $(BUILD_DIRECTORY)/electron-$(TARGET_PLATFORM)-$(TARGET_ARCH)-dependencies
-	./scripts/build/dependencies-bower.sh -p -x $|
-
 $(BUILD_DIRECTORY)/electron-$(TARGET_PLATFORM)-$(APPLICATION_VERSION)-$(TARGET_ARCH)-app: \
 	$(BUILD_DIRECTORY)/electron-$(TARGET_PLATFORM)-$(TARGET_ARCH)-dependencies/node_modules \
-	$(BUILD_DIRECTORY)/electron-$(TARGET_PLATFORM)-$(TARGET_ARCH)-dependencies/bower_components \
 	| $(BUILD_DIRECTORY)
 	./scripts/build/electron-create-resources-app.sh -s . -o $@ \
 		-v $(APPLICATION_VERSION) \
 		-f "$(APPLICATION_FILES)"
-	$(foreach prerequisite,$^,$(call execute-command,cp -RLf $(prerequisite) $@))
+	cp -Rlf $< $@
 
 $(BUILD_DIRECTORY)/electron-$(TARGET_PLATFORM)-$(APPLICATION_VERSION)-$(TARGET_ARCH)-app.asar: \
 	$(BUILD_DIRECTORY)/electron-$(TARGET_PLATFORM)-$(APPLICATION_VERSION)-$(TARGET_ARCH)-app \
@@ -496,7 +491,6 @@ electron-develop:
 		-v "$(ELECTRON_VERSION)" \
 		-t electron \
 		-s "$(TARGET_PLATFORM)"
-	./scripts/build/dependencies-bower.sh
 
 help:
 	@echo "Available targets: $(TARGETS)"
@@ -520,6 +514,5 @@ clean:
 
 distclean: clean
 	rm -rf node_modules
-	rm -rf bower_components
 
 .DEFAULT_GOAL = help
diff --git a/appveyor.yml b/appveyor.yml
@@ -21,7 +21,7 @@ environment:
 
 install:
   - ps: Install-Product node $env:nodejs_version x64
-  - npm install -g bower rimraf asar
+  - npm install -g rimraf asar
   - choco install nsis -version 2.51
   - choco install jq
   - choco install curl

diff --git a/bower.json b/bower.json
diff --git a/docs/RUNNING-LOCALLY.md b/docs/RUNNING-LOCALLY.md
@@ -10,7 +10,6 @@ Prerequisites
 ### Common
 
 - [NodeJS](https://nodejs.org) (at least v6)
-- [Bower](http://bower.io)
 - [Python](https://www.python.org)
 - [jq](https://stedolan.github.io/jq/)
 - [Asar](https://github.com/electron/asar)

diff --git a/lib/gui/index.html b/lib/gui/index.html
@@ -11,17 +11,6 @@
     <!-- See https://github.com/visionmedia/debug#browser-support -->
     <script>window.localStorage.debug = '*';</script>
 
-    <script>
-      window._trackJs = {
-        token: '032448bc3d9e4cffb1e9b43d29d6c142',
-        application: 'etcher',
-        console: {
-          error: false
-        }
-      };
-    </script>
-    <script src="../../node_modules/trackjs/tracker.js"></script>
-
     <script src="./app.js"></script>
   </head>
   <body ng-app="Etcher">

diff --git a/lib/gui/modules/analytics.js b/lib/gui/modules/analytics.js
@@ -22,106 +22,46 @@
 
 const _ = require('lodash');
 const angular = require('angular');
-const username = require('username');
-const isRunningInAsar = require('electron-is-running-in-asar');
+const resinCorvus = require('resin-corvus/browser');
 const errors = require('../../shared/errors');
-const os = require('os');
 const packageJSON = require('../../../package.json');
-const arch = require('arch');
-const utils = require('../../shared/utils');
 
-// Force Mixpanel snippet to load Mixpanel locally
-// instead of using a CDN for performance reasons
-window.MIXPANEL_CUSTOM_LIB_URL = '../../bower_components/mixpanel/mixpanel.js';
-
-require('../../../bower_components/mixpanel/mixpanel-jslib-snippet.js');
-require('../../../bower_components/angular-mixpanel/src/angular-mixpanel');
 const MODULE_NAME = 'Etcher.Modules.Analytics';
 const analytics = angular.module(MODULE_NAME, [
-  'analytics.mixpanel',
   require('../models/settings')
 ]);
 
-/**
- * @summary Get host architecture
- * @function
- * @private
- *
- * @description
- * We need this because node's os.arch() returns the process architecture
- * See: https://github.com/nodejs/node-v0.x-archive/issues/2862
- *
- * @returns {String} Host architecture
- *
- * @example
- * if (getHostArchitecture() === 'x64') {
- *   console.log('Host architecture is x64');
- * }
- */
-const getHostArchitecture = () => {
-  return _.includes([ 'ia32', 'x64' ], process.arch) ? _.replace(arch(), 'x86', 'ia32') : process.arch;
-};
-
-// Mixpanel integration
-// https://github.com/kuhnza/angular-mixpanel
-
-analytics.config(($mixpanelProvider) => {
-  $mixpanelProvider.apiKey('63e5fc4563e00928da67d1226364dd4c');
-
-  $mixpanelProvider.superProperties({
-    electron: process.versions.electron,
-    node: process.version,
-    arch: process.arch,
-    version: packageJSON.version,
-    osPlatform: os.platform(),
-    hostArch: getHostArchitecture(),
-    osRelease: os.release(),
-    cpuCores: os.cpus().length,
-    totalMemory: os.totalmem(),
-    startFreeMemory: os.freemem()
-  });
-});
-
-// TrackJS integration
-// http://docs.trackjs.com/tracker/framework-integrations
-
-analytics.run(($window) => {
-
-  // Don't configure TrackJS when
-  // running inside the test suite
-  if (window.mocha) {
-    return;
-  }
-
-  $window.trackJs.configure({
-    userId: username.sync(),
-    version: packageJSON.version
+analytics.config(() => {
+  resinCorvus.install({
+    services: {
+      sentry: process.env.SENTRY_DSN,
+      mixpanel: process.env.MIXPANEL_TOKEN
+    },
+    options: {
+      release: packageJSON.version
+    }
   });
 });
 
-analytics.service('AnalyticsService', function($log, $window, $mixpanel, SettingsModel) {
+analytics.service('AnalyticsService', function($log, SettingsModel) {
 
   /**
    * @summary Log a debug message
    * @function
    * @public
    *
    * @description
-   * This function sends the debug message to TrackJS only.
+   * This function sends the debug message to error reporting / product analytics services.
    *
    * @param {String} message - message
    *
    * @example
    * AnalyticsService.log('Hello World');
    */
   this.logDebug = (message) => {
-    const debugMessage = `${new Date()} ${message}`;
-
-    if (SettingsModel.get('errorReporting') && isRunningInAsar()) {
-      $window.trackJs.console.debug(debugMessage);
+    if (SettingsModel.get('errorReporting')) {
+      resinCorvus.logDebug(message);
     }
-
-    $log.debug(debugMessage);
   };
 
   /**
@@ -130,7 +70,7 @@ analytics.service('AnalyticsService', function($log, $window, $mixpanel, Setting
    * @public
    *
    * @description
-   * This function sends the debug message to TrackJS and Mixpanel.
+   * This function sends the debug message to error reporting / product analytics services.
    *
    * @param {String} message - message
    * @param {Object} [data] - event data
@@ -141,21 +81,9 @@ analytics.service('AnalyticsService', function($log, $window, $mixpanel, Setting
    * });
    */
   this.logEvent = (message, data) => {
-    const debugData = utils.hideAbsolutePathsInObject(utils.makeFlatStartCaseObject(data));
-
-    if (SettingsModel.get('errorReporting') && isRunningInAsar()) {
-      $mixpanel.track(message, debugData);
+    if (SettingsModel.get('errorReporting')) {
+      resinCorvus.logEvent(message, data);
     }
-
-    const debugMessage = _.attempt(() => {
-      if (debugData) {
-        return `${message} (${JSON.stringify(debugData)})`;
-      }
-
-      return message;
-    });
-
-    this.logDebug(debugMessage);
   };
 
   /**
@@ -164,7 +92,7 @@ analytics.service('AnalyticsService', function($log, $window, $mixpanel, Setting
    * @public
    *
    * @description
-   * This function logs an exception in TrackJS.
+   * This function logs an exception to a error reporting services.
    *
    * @param {Error} exception - exception
    *
@@ -174,13 +102,12 @@ analytics.service('AnalyticsService', function($log, $window, $mixpanel, Setting
   this.logException = (exception) => {
     if (_.every([
       SettingsModel.get('errorReporting'),
-      isRunningInAsar(),
       errors.shouldReport(exception)
     ])) {
-      $window.trackJs.track(exception);
+      resinCorvus.logException(exception);
+    } else {
+      $log.error(exception);
     }
-
-    $log.error(exception);
   };
 
 });