Test sentry on CI

.gitignore

@@ -17,4 +17,9 @@ gsender_server_log.txt
 
 # Sentry Auth Token
 .env.sentry-build-plugin
-cypress.env.json
+cypress.env.json
+# Sentry Config File
+.env.sentry-build-plugin
+
+# Sentry Config File
+.env.sentry-build-plugin

package.json

@@ -171,7 +171,7 @@
     "@mui/material": "^5.9.2",
     "@octokit/core": "^3.6.0",
     "@react-pdf/renderer": "3.0.0",
-    "@sentry/electron": "^4.11.0",
+    "@sentry/electron": "^4.14.0",
     "@sentry/react": "^7.72.0",
     "@serialport/parser-readline": "^11.0.0",
     "@sienci/avrgirl-arduino": "^5.0.3",

src/app/index.jsx

@@ -39,7 +39,7 @@ import LanguageDetector from 'i18next-browser-languagedetector';
 import XHR from 'i18next-xhr-backend';
 import { TRACE, DEBUG, INFO, WARN, ERROR } from 'universal-logger';
 import { Provider as ReduxProvider } from 'react-redux';
-//import * as Sentry from '@sentry/react';
+import * as Sentry from '@sentry/react';
 
 import { Provider as GridSystemProvider } from 'app/components/GridSystem';
 import rootSaga from 'app/sagas';
@@ -65,15 +65,13 @@ import PopUpWidget from './containers/PopUpWidget';
 import './styles/vendor.styl';
 import './styles/app.styl';
 
-// import * as Sentry from '@sentry/react';
-
-// Sentry.init({
-//     dsn: 'https://c09ff263997c4a47ba22b3c948f19734@o558751.ingest.sentry.io/5692684',
-//     integrations: [
-//         new Sentry.BrowserTracing(),
-//         new Sentry.Replay()
-//     ],
-// });
+Sentry.init({
+    dsn: 'https://c09ff263997c4a47ba22b3c948f19734@o558751.ingest.sentry.io/5692684',
+    integrations: [
+        new Sentry.BrowserTracing(),
+        new Sentry.Replay()
+    ],
+});
 
 const renderPage = () => {
     const container = document.createElement('div');

src/main.js

@@ -30,7 +30,7 @@ import isOnline from 'is-online';
 import log from 'electron-log';
 import path from 'path';
 import fs from 'fs';
-// import * as Sentry from '@sentry/electron/main';
+import * as Sentry from '@sentry/electron/main';
 
 import WindowManager from './electron-app/WindowManager';
 import launchServer from './server-cli';
@@ -44,7 +44,7 @@ let hostInformation = {};
 let grblLog = log.create('grbl');
 let logPath;
 
-// Sentry.init({ dsn: 'https://c09ff263997c4a47ba22b3c948f19734@o558751.ingest.sentry.io/5692684' });
+Sentry.init({ dsn: 'https://c09ff263997c4a47ba22b3c948f19734@o558751.ingest.sentry.io/5692684' });
 
 const main = () => {
     // https://github.com/electron/electron/blob/master/docs/api/app.md#apprequestsingleinstancelock
@@ -134,7 +134,7 @@ const main = () => {
                 }
             }
 
-            const { address, port, requestedHost, kiosk } = { ...res };
+            const { address, port, kiosk } = { ...res };
             log.info(`Returned - http://${address}:${port}`);
             hostInformation = {
                 address,
@@ -201,12 +201,12 @@ const main = () => {
                 if ('type' in error) {
                     log.transports.file.level = 'error';
                 }
-                if(error.type.includes('GRBL_HAL')) {
+
+                if (error.type.includes('GRBL_HAL')) {
                     (error.type === 'GRBL_HAL_ERROR') ? grblLog.error(`GRBL_HAL_ERROR:Error ${error.code} - ${error.description} Line ${error.lineNumber}: "${error.line.trim()}" Origin- ${error.origin.trim()}`) : grblLog.error(`GRBL_HAL_ALARM:Alarm ${error.code} - ${error.description}`);
                 } else {
                     (error.type === 'GRBL_ERROR') ? grblLog.error(`GRBL_ERROR:Error ${error.code} - ${error.description} Line ${error.lineNumber}: "${error.line.trim()}" Origin- ${error.origin.trim()}`) : grblLog.error(`GRBL_ALARM:Alarm ${error.code} - ${error.description}`);
                 }
-
             });
 
             ipcMain.on('clipboard', (channel, text) => {

src/package.json

@@ -14,7 +14,7 @@
   },
   "main": "main.js",
   "dependencies": {
-    "@sentry/electron": "^4.11.0",
+    "@sentry/electron": "^4.14.0",
     "@sentry/react": "^7.72.0",
     "@serialport/parser-readline": "^11.0.0",
     "@sienci/avrgirl-arduino": "^5.0.3",

src/server/index.js

@@ -39,7 +39,6 @@ import size from 'lodash/size';
 import trimEnd from 'lodash/trimEnd';
 import uniqWith from 'lodash/uniqWith';
 import webappengine from 'webappengine';
-// import * as Sentry from '@sentry/node';
 
 import settings from './config/settings';
 import app from './app';
@@ -52,11 +51,6 @@ import urljoin from './lib/urljoin';
 
 const log = logger('init');
 
-// Sentry.init({
-//     dsn: 'https://c09ff263997c4a47ba22b3c948f19734@o558751.ingest.sentry.io/5692684',
-//     tracesSampleRate: 0.3,
-// });
-
 const createServer = (options, callback) => {
     options = { ...options };
 

webpack.config.app.production.js

@@ -13,7 +13,7 @@ const stylusLoader = require('stylus-loader');
 const TerserPlugin = require('terser-webpack-plugin');
 const webpack = require('webpack');
 const ManifestPlugin = require('webpack-manifest-plugin');
-// const { sentryWebpackPlugin } = require('@sentry/webpack-plugin');
+const { sentryWebpackPlugin } = require('@sentry/webpack-plugin');
 
 const babelConfig = require('./babel.config');
 const buildConfig = require('./build.config');
@@ -42,7 +42,8 @@ module.exports = {
     cache: true,
     target: 'web',
     context: path.resolve(__dirname, 'src/app'),
-    devtool: 'cheap-module-source-map',
+    devtool: 'source-map',
+    // devtool: 'cheap-module-source-map',
     entry: {
         polyfill: [
             path.resolve(__dirname, 'src/app/polyfill/index.js')
@@ -58,8 +59,8 @@ module.exports = {
     },
     output: {
         path: path.resolve(__dirname, 'dist/gsender/app'),
-        chunkFilename: `[name].[chunkhash].bundle.js?_=${timestamp}`,
-        filename: `[name].[chunkhash].bundle.js?_=${timestamp}`,
+        chunkFilename: '[name].[chunkhash].bundle.js',
+        filename: '[name].[chunkhash].bundle.js',
         publicPath: publicPath
     },
     module: {
@@ -243,14 +244,14 @@ module.exports = {
             meta: {
                 'viewport': 'width=device-width, initial-scale=1, maximum-scale=1',
             }
+        }),
+        sentryWebpackPlugin({
+            org: process.env.SENTRY_ORG,
+            project: process.env.SENTRY_PROJECT,
+            authToken: process.env.SENTRY_AUTH_TOKEN,
+            urlPrefix: publicPath,
+            telemetry: false
         })],
-    // sentryWebpackPlugin({
-    //     org: process.env.SENTRY_ORG,
-    //     project: process.env.SENTRY_PROJECT,
-    //     authToken: process.env.SENTRY_AUTH_TOKEN,
-    //     urlPrefix: publicPath,
-    //     telemetry: false
-    // })],
     resolve: {
         modules: [
             path.resolve(__dirname, 'src'),

webpack.config.server.production.js

@@ -6,7 +6,7 @@ const TerserPlugin = require('terser-webpack-plugin');
 const HtmlWebpackPlugin = require('html-webpack-plugin');
 const webpack = require('webpack');
 const nodeExternals = require('webpack-node-externals');
-// const { sentryWebpackPlugin } = require('@sentry/webpack-plugin');
+const { sentryWebpackPlugin } = require('@sentry/webpack-plugin');
 
 const babelConfig = require('./babel.config');
 const pkg = require('./package.json');
@@ -30,7 +30,8 @@ const buildVersion = pkg.version;
 
 module.exports = {
     mode: 'production',
-    devtool: 'cheap-module-source-map',
+    devtool: 'source-map',
+    // devtool: 'cheap-module-source-map',
     target: 'node', // ignore built-in modules like path, fs, etc.
     context: path.resolve(__dirname, 'src/server'),
     entry: {
@@ -62,12 +63,12 @@ module.exports = {
             template: path.resolve(__dirname, 'index.html'),
             chunksSortMode: 'dependency' // Sort chunks by dependency
         }),
-        // sentryWebpackPlugin({
-        //     org: process.env.SENTRY_ORG,
-        //     project: process.env.SENTRY_PROJECT,
-        //     authToken: process.env.SENTRY_AUTH_TOKEN,
-        //     telemetry: false
-        // }),
+        sentryWebpackPlugin({
+            org: process.env.SENTRY_ORG,
+            project: process.env.SENTRY_PROJECT,
+            authToken: process.env.SENTRY_AUTH_TOKEN,
+            telemetry: false
+        }),
     ],
     module: {
         rules: [