Merge pull request #23 from eth-cscs/1548-remove-sentry-dep

1548 remove Sentry dependency

Author: francescopagnamenta

.gitignore

@@ -29,6 +29,11 @@ sessions/
 node_modules
 node_modules*
 
-# sentry
-.sentryclirc
-.env.sentry-build-plugin
+
+# Yarn Berry
+.yarn/*
+!.yarn/patches
+!.yarn/plugins
+!.yarn/releases
+!.yarn/sdks
+!.yarn/versions

.yarn/install-state.gz

@@ -1 +1,5 @@
-nodeLinker: node-modules
+nodeLinker: node-modules
+packageExtensions:
+  "remix-keycloak@*":
+    peerDependencies:
+      "@remix-run/react": "^2.16.0"

.yarnrc.yml

@@ -5,8 +5,6 @@
   SPDX-License-Identifier: BSD-3-Clause
 *************************************************************************/
 
-import * as Sentry from '@sentry/remix'
-import { replayIntegration } from '@sentry/remix'
 import { hydrateRoot } from 'react-dom/client'
 import { startTransition, StrictMode, useEffect } from 'react'
 import { RemixBrowser, useLocation, useMatches } from '@remix-run/react'
@@ -17,35 +15,10 @@ declare global {
     ENV: {
       APP_VERSION: string
       ENVIRONMENT: string
-      SENTRY_ACTIVE: boolean
-      SENTRY_DSN: string
-      SENTRY_DEBUG: boolean
-      SENTRY_TRACES_SAMPLE_RATE: number
     }
   }
 }
 
-// Initialize sentry
-if (window.ENV.SENTRY_ACTIVE) {
-  Sentry.init({
-    dsn: window.ENV.SENTRY_DSN,
-    debug: window.ENV.SENTRY_DEBUG,
-    environment: window.ENV.ENVIRONMENT,
-    release: window.ENV.APP_VERSION,
-    tracesSampleRate: parseFloat(window.ENV.SENTRY_TRACES_SAMPLE_RATE + ''),
-    replaysSessionSampleRate: 0.1,
-    replaysOnErrorSampleRate: 1,
-    integrations: [
-      Sentry.browserTracingIntegration({
-        useEffect,
-        useLocation,
-        useMatches,
-      }),
-      replayIntegration(),
-    ],
-  })
-}
-
 startTransition(() => {
   hydrateRoot(
     document,

app/configs/sentry.config.ts

@@ -4,31 +4,12 @@
   Please, refer to the LICENSE file in the root directory.
   SPDX-License-Identifier: BSD-3-Clause
 *************************************************************************/
-import * as Sentry from '@sentry/remix'
 import { PassThrough } from 'node:stream'
 import { isbot } from 'isbot'
 import { RemixServer } from '@remix-run/react'
 import { renderToPipeableStream } from 'react-dom/server'
 import { createReadableStreamFromReadable } from '@remix-run/node'
 import type { AppLoadContext, EntryContext } from '@remix-run/node'
-// configs
-import base from './configs/base.config'
-import sentry from './configs/sentry.config'
-
-export function handleError(error: any, { request }: any) {
-  Sentry.captureRemixServerException(error, 'remix.server', request)
-}
-
-// Sentry initialization
-if (sentry.active) {
-  Sentry.init({
-    dsn: sentry.dsn,
-    debug: sentry.debug,
-    environment: base.environment,
-    release: base.appVersion,
-    tracesSampleRate: parseFloat(sentry.tracesSampleRate),
-  })
-}
 
 const ABORT_DELAY = 5_000
 

app/entry.client.tsx

@@ -7,7 +7,6 @@
 
 import { ValidationError } from 'yup'
 import { StatusCodes } from 'http-status-codes'
-import { captureException } from '@sentry/remix'
 import { json, MaxPartSizeExceededError } from '@remix-run/node'
 // types
 import { ErrorType } from '~/types/error'
@@ -67,8 +66,6 @@ export const handleErrorResponse = (
   error: any,
   handlingThreshold: StatusCodes | null = StatusCodes.INTERNAL_SERVER_ERROR,
 ) => {
-  // Capture exception with remix
-  captureException(error)
   // Validation http error
   if (error instanceof ValidationError) {
     return handleValidationErrorResponse(error)

app/entry.server.tsx

@@ -16,12 +16,10 @@ import {
   useLoaderData,
   useRouteError,
 } from '@remix-run/react'
-import { captureRemixErrorBoundaryError } from '@sentry/remix'
 // styles
 import stylesheet from '~/tailwind.css?url'
 // configs
 import base from './configs/base.config'
-import sentry from './configs/sentry.config'
 // pages
 import ErrorPage from './components/pages/ErrorPage'
 import logger from './logger/logger'
@@ -34,10 +32,6 @@ export async function loader({ context }: LoaderFunctionArgs) {
       APP_NAME: base.appName,
       APP_VERSION: base.appVersion,
       ENVIRONMENT: base.environment,
-      SENTRY_ACTIVE: sentry.active,
-      SENTRY_DSN: sentry.dsn,
-      SENTRY_DEBUG: sentry.debug,
-      SENTRY_TRACES_SAMPLE_RATE: sentry.tracesSampleRate,
     },
   })
 }
@@ -94,7 +88,6 @@ export default function App() {
 export function ErrorBoundary() {
   const error = useRouteError()
   logger.error(error)
-  captureRemixErrorBoundaryError(error)
   return (
     <Document title='FirecREST Web UI - v2'>
       <ErrorPage />

app/helpers/response-helper.ts

@@ -6,7 +6,6 @@
 *************************************************************************/
 
 import { useLoaderData, useRouteError } from '@remix-run/react'
-import { captureRemixErrorBoundaryError } from '@sentry/remix'
 import type { LoaderFunction, LoaderFunctionArgs } from '@remix-run/node'
 // loggers
 import logger from '~/logger/logger'
@@ -77,6 +76,5 @@ export default function AppIndexRoute() {
 export function ErrorBoundary() {
   const error = useRouteError()
   logger.error(error)
-  captureRemixErrorBoundaryError(error)
   return <ErrorView error={error} />
 }

app/root.tsx

@@ -5,7 +5,6 @@
   SPDX-License-Identifier: BSD-3-Clause
 *************************************************************************/
 
-import { captureRemixErrorBoundaryError } from '@sentry/remix'
 import { useLoaderData, useRouteError } from '@remix-run/react'
 import type { LoaderFunction, LoaderFunctionArgs } from '@remix-run/node'
 // types
@@ -60,6 +59,5 @@ export default function AppComputeIndexRoute() {
 export function ErrorBoundary() {
   const error = useRouteError()
   logger.error(error)
-  captureRemixErrorBoundaryError(error)
   return <ErrorView error={error} />
 }