fix: Revert sentry changes (#904)

.gitignore

@@ -10,4 +10,3 @@ main.map
 Certificates.p12
 .cache/
 .idea/
-sentry-version.js

CONTRIBUTING.md

@@ -153,10 +153,6 @@ Instructions will be given by the GitHub Bot when you make a pull request.
 Just use the GitHub issue tracker to submit your bug reports and feature
 requests. If you are submitting a bug report, please follow the [issue template](https://github.com/appium/appium-desktop/issues/new).
 
-### Sentry
-
-Appium Desktop has crash logs reported to it using [Sentry](https://sentry.io/appium/appiumdesktop/). This requires permission to access.
-
 ### Localization
 
 Appium Desktop uses [i18next](https://www.i18next.com) library to manage its translations. All translatable resources are stored in `assets/locales/en/translation.json` file. The list of supported languages can be found (or altered) in `app/configs/app.config.js` module. Please read through i18next API documentation to get more information on how to manage the localized resources:

app/main.js

@@ -5,8 +5,6 @@ import { setSavedEnv } from './main/helpers';
 import rebuildMenus from './main/menus';
 import shellEnv from 'shell-env';
 import fixPath from 'fix-path';
-import { initSentry } from './shared/sentry';
-import { promptUser } from './main/sentry-permission-prompt';
 import settings from './shared/settings';
 
 let mainWindow = null;
@@ -27,9 +25,6 @@ if (!isDev) {
 }
 setSavedEnv();
 
-// Enable Sentry crash report logging
-initSentry();
-
 app.on('window-all-closed', () => {
   app.quit();
 });
@@ -100,7 +95,5 @@ app.on('ready', async () => {
 
   rebuildMenus(mainWindow);
 
-  promptUser();
-
   initializeIpc(mainWindow);
 });

app/main/appium.js

@@ -379,12 +379,6 @@ function connectSaveEnv () {
   });
 }
 
-function connectServerErrorBackdoor () {
-  ipcMain.on('appium-force-nodejs-error', () => {
-    throw new Error('A NodeJS error was intentionally thrown');
-  });
-}
-
 export function initializeIpc (win) {
   // listen for 'start-server' from the renderer
   connectStartServer(win);
@@ -403,7 +397,6 @@ export function initializeIpc (win) {
   connectOpenConfig(win);
   connectGetEnv();
   connectSaveEnv();
-  connectServerErrorBackdoor();
 
   setTimeout(checkNewUpdates, 10000);
 }

app/renderer/components/Session/CloudProviderSelector.js

@@ -31,7 +31,7 @@ export default class CloudProviderSelector extends Component {
   }
 
   render () {
-    const {t, isAddingCloudProvider, stopAddCloudProvider, visibleProviders} = this.props;
+    const {t, isAddingCloudProvider, stopAddCloudProvider, visibleProviders = []} = this.props;
     const providersGrid = _.chunk(_.keys(CloudProviders), 2); // Converts list of providers into list of pairs of providers
     const footer = [<Button key="back" type="primary" onClick={stopAddCloudProvider}>{t('Done')}</Button>];
 

app/renderer/components/StartServer/shared.js

@@ -1,5 +1,4 @@
 import PropTypes from 'prop-types';
-import { ipcRenderer } from 'electron';
 import { DEFAULT_ARGS } from '../../reducers/StartServer';
 
 export const propTypes = {
@@ -17,16 +16,6 @@ export function updateArg (evt) {
   const {updateArgs} = this.props;
   let argName = evt.target.name;
 
-  // Backdoor for testing purposes only. This forces an error to occur so
-  // that we can test errors on sentry
-  if (evt.target.value) {
-    if (evt.target.value.toLowerCase() === 'force_browser_error') {
-      throw new Error(`Browser error intentionally thrown`);
-    } else if (evt.target.value && evt.target.value.toLowerCase() === 'force_nodejs_error') {
-      ipcRenderer.send('appium-force-nodejs-error');
-    }
-  }
-
   let newVal;
   switch (evt.target.type) {
     case 'checkbox':

app/renderer/index.js

@@ -3,14 +3,11 @@ import { render } from 'react-dom';
 import Root from './containers/Root';
 import { AppContainer } from 'react-hot-loader';
 import Store from './store/configureStore';
-import { initSentry } from '../shared/sentry';
 
 const { history, configureStore } = Store;
 
 const store = configureStore();
 
-initSentry();
-
 render(
   <AppContainer>
     <Root store={store} history={history} />

ci-jobs/templates/package.yml

@@ -16,7 +16,6 @@ jobs:
       E2E_TIMEOUT: 600000
       CI: true
       CSC_IDENTITY_AUTO_DISCOVERY: true
-      SENTRY_ORG: appium
     steps:
     - task: NodeTool@0
       inputs:
@@ -31,4 +30,3 @@ jobs:
         GH_TOKEN: $(GH_TOKEN)
         CSC_LINK: $(CSC_LINK)
         CSC_KEY_PASSWORD: $(CSC_KEY_PASSWORD)
-        SENTRY_AUTH_TOKEN: $(SENTRY_AUTH_TOKEN)