groot/automatisch
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #1695 from automatisch/sentry-enhancement

Browse Source 
chore: Do not enable sentry for dev and test environments
This commit is contained in:
Ömer Faruk Aydın
2024-03-05 15:53:36 +01:00
committed by GitHub
parent ee9433261b 35ea18a117
commit f4d8d909b0
1 changed files with 9 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
packages/backend/src/helpers/sentry.ee.js
View File

@@ -3,10 +3,13 @@ import * as Tracing from '@sentry/tracing';
import appConfig from '../config/app.js'; import appConfig from '../config/app.js';
const isSentryEnabled = !!appConfig.sentryDsn; const isSentryEnabled = () => {
if (appConfig.isDev || appConfig.isTest) return false;
return !!appConfig.sentryDsn;
};
export function init(app) { export function init(app) {
if (!isSentryEnabled) return; if (!isSentryEnabled()) return;
return Sentry.init({ return Sentry.init({
enabled: !!appConfig.sentryDsn, enabled: !!appConfig.sentryDsn,
@@ -21,19 +24,19 @@ export function init(app) {
} }
export function attachRequestHandler(app) { export function attachRequestHandler(app) {
if (!isSentryEnabled) return; if (!isSentryEnabled()) return;
app.use(Sentry.Handlers.requestHandler()); app.use(Sentry.Handlers.requestHandler());
} }
export function attachTracingHandler(app) { export function attachTracingHandler(app) {
if (!isSentryEnabled) return; if (!isSentryEnabled()) return;
app.use(Sentry.Handlers.tracingHandler()); app.use(Sentry.Handlers.tracingHandler());
} }
export function attachErrorHandler(app) { export function attachErrorHandler(app) {
if (!isSentryEnabled) return; if (!isSentryEnabled()) return;
app.use( app.use(
Sentry.Handlers.errorHandler({ Sentry.Handlers.errorHandler({
@@ -46,7 +49,7 @@ export function attachErrorHandler(app) {
} }
export function captureException(exception, captureContext) { export function captureException(exception, captureContext) {
if (!isSentryEnabled) return; if (!isSentryEnabled()) return;
return Sentry.captureException(exception, captureContext); return Sentry.captureException(exception, captureContext);
} }