refactor: remove unused logrocket (#4924)

novuhq · Dec 1, 2023 · c155ae1 · c155ae1
1 parent 0dcb0c6
commit c155ae1
Show file tree

Hide file tree

Showing 8 changed files with 6 additions and 112 deletions.
diff --git a/.cspell.json b/.cspell.json
@@ -399,7 +399,6 @@
     "nunito",
     "codesee",
     "ehlo",
-    "logrocket",
     "backoff",
     "isequal",
     "Scriptable",

diff --git a/.github/workflows/prod-deploy-web.yml b/.github/workflows/prod-deploy-web.yml
@@ -48,7 +48,6 @@ jobs:
           echo REACT_APP_SENTRY_DSN="https://[email protected]/6250907" >> .env
           echo REACT_APP_ENVIRONMENT=production >> .env
           echo REACT_APP_SEGMENT_KEY=${{ secrets.WEB_SEGMENT_KEY }} >> .env
-          echo REACT_APP_LOGROCKET_ID=${{ secrets.LOGROCKET_ID }} >> .env
           echo REACT_APP_MAIL_SERVER_DOMAIN="eu.inbound-mail.novu.co" >> .env
           echo REACT_APP_LAUNCH_DARKLY_CLIENT_SIDE_ID=${{ secrets.PROD_LAUNCH_DARKLY_CLIENT_SIDE_ID }} >> .env
 
@@ -65,7 +64,6 @@ jobs:
           REACT_APP_WEBHOOK_URL: https://eu.webhook.novu.co
           REACT_APP_WIDGET_EMBED_PATH: https://eu.embed.novu.co/embed.umd.min.js
           REACT_APP_NOVU_APP_ID: ${{ secrets.NOVU_APP_ID }}
-          REACT_APP_LOGROCKET_ID: ${{ secrets.LOGROCKET_ID }}
           REACT_APP_SENTRY_DSN: https://[email protected]/6250907
           REACT_APP_ENVIRONMENT: production
           REACT_APP_MAIL_SERVER_DOMAIN: eu.inbound-mail.novu.co
@@ -128,7 +126,6 @@ jobs:
           echo REACT_APP_SENTRY_DSN="https://[email protected]/6250907" >> .env
           echo REACT_APP_ENVIRONMENT=production >> .env
           echo REACT_APP_SEGMENT_KEY=${{ secrets.WEB_SEGMENT_KEY }} >> .env
-          echo REACT_APP_LOGROCKET_ID=${{ secrets.LOGROCKET_ID }} >> .env
           echo REACT_APP_MAIL_SERVER_DOMAIN="inbound-mail.novu.co" >> .env
           echo REACT_APP_LAUNCH_DARKLY_CLIENT_SIDE_ID=${{ secrets.PROD_LAUNCH_DARKLY_CLIENT_SIDE_ID }} >> .env
 
@@ -145,7 +142,6 @@ jobs:
           REACT_APP_WEBHOOK_URL: https://webhook.novu.co
           REACT_APP_WIDGET_EMBED_PATH: https://embed.novu.co/embed.umd.min.js
           REACT_APP_NOVU_APP_ID: ${{ secrets.NOVU_APP_ID }}
-          REACT_APP_LOGROCKET_ID: ${{ secrets.LOGROCKET_ID }}
           REACT_APP_SENTRY_DSN: https://[email protected]/6250907
           REACT_APP_ENVIRONMENT: production
           REACT_APP_MAIL_SERVER_DOMAIN: inbound-mail.novu.co

diff --git a/apps/web/.env b/apps/web/.env
@@ -13,7 +13,6 @@ REACT_APP_SENTRY_DSN=
 REACT_APP_BLUEPRINTS_API_URL=
 REACT_APP_MAIL_SERVER_DOMAIN=
 REACT_APP_LAUNCH_DARKLY_CLIENT_SIDE_ID=
-REACT_APP_LOGROCKET_ID=
 IS_TEMPLATE_STORE_ENABLED=
 IS_MULTI_PROVIDER_CONFIGURATION_ENABLED=
 IS_MULTI_TENANCY_ENABLED=
diff --git a/apps/web/package.json b/apps/web/package.json
@@ -87,8 +87,6 @@
     "lodash.get": "^4.3.2",
     "lodash.isequal": "^4.5.0",
     "lodash.set": "^4.3.2",
-    "logrocket": "^3.0.1",
-    "logrocket-react": "^5.0.1",
     "polished": "^4.1.3",
     "react": "^17.0.1",
     "react-ace": "^9.4.3",

diff --git a/apps/web/src/App.tsx b/apps/web/src/App.tsx
@@ -3,14 +3,11 @@ import { QueryClient, QueryClientProvider } from '@tanstack/react-query';
 import { HelmetProvider } from 'react-helmet-async';
 import { Route, Routes, BrowserRouter } from 'react-router-dom';
 import { withLDProvider } from 'launchdarkly-react-client-sdk';
-import LogRocket from 'logrocket';
-import setupLogRocketReact from 'logrocket-react';
 import { Integrations } from '@sentry/tracing';
 import { library } from '@fortawesome/fontawesome-svg-core';
 import { far } from '@fortawesome/free-regular-svg-icons';
 import { fas } from '@fortawesome/free-solid-svg-icons';
 
-import packageJson from '../package.json';
 import { AuthProvider } from './components/providers/AuthProvider';
 import { applyToken, getToken } from './hooks';
 import { ActivitiesPage } from './pages/activities/ActivitiesPage';
@@ -27,7 +24,7 @@ import { PasswordResetPage } from './pages/auth/PasswordResetPage';
 import { AppLayout } from './components/layout/AppLayout';
 import { MembersInvitePage } from './pages/invites/MembersInvitePage';
 import CreateOrganizationPage from './pages/auth/CreateOrganizationPage';
-import { ENV, LAUNCH_DARKLY_CLIENT_SIDE_ID, SENTRY_DSN, CONTEXT_PATH, LOGROCKET_ID } from './config';
+import { ENV, LAUNCH_DARKLY_CLIENT_SIDE_ID, SENTRY_DSN, CONTEXT_PATH } from './config';
 import { PromoteChangesPage } from './pages/changes/PromoteChangesPage';
 import { LinkVercelProjectPage } from './pages/partner-integrations/LinkVercelProjectPage';
 import { ROUTES } from './constants/routes.enum';
@@ -64,32 +61,6 @@ import { VariantsPage } from './pages/templates/components/VariantsPage';
 
 library.add(far, fas);
 
-if (LOGROCKET_ID && window !== undefined) {
-  LogRocket.init(LOGROCKET_ID, {
-    release: packageJson.version,
-    rootHostname: 'novu.co',
-    console: {
-      shouldAggregateConsoleErrors: true,
-    },
-    network: {
-      requestSanitizer: (request) => {
-        // if the url contains token 'ignore' it
-        if (request.url.toLowerCase().indexOf('token') !== -1) {
-          // ignore the request response pair
-          return null;
-        }
-
-        // remove Authorization header from logrocket
-        request.headers.Authorization = undefined;
-
-        // otherwise log the request normally
-        return request;
-      },
-    },
-  });
-  setupLogRocketReact(LogRocket);
-}
-
 if (SENTRY_DSN) {
   Sentry.init({
     dsn: SENTRY_DSN,
@@ -122,29 +93,9 @@ if (SENTRY_DSN) {
      */
     tracesSampleRate: 1.0,
     beforeSend(event: Sentry.Event) {
-      const logRocketSession = LogRocket.sessionURL;
-
-      if (logRocketSession !== null || (event as string) !== '' || event !== undefined) {
-        /*
-         * Must ignore the next line as this variable could be null but
-         * can not be null because of the check in the if statement above.
-         */
-        // eslint-disable-next-line @typescript-eslint/ban-ts-comment
-        // @ts-expect-error
-        event.extra.LogRocket = logRocketSession;
-
-        return event;
-      } else {
-        return event;
-      } //else
+      return event;
     },
   });
-
-  LogRocket.getSessionURL((sessionURL) => {
-    Sentry.configureScope((scope) => {
-      scope.setExtra('sessionURL', sessionURL);
-    });
-  });
 }
 
 const defaultQueryFn = async ({ queryKey }: { queryKey: string }) => {

diff --git a/apps/web/src/components/layout/components/HeaderNav.tsx b/apps/web/src/components/layout/components/HeaderNav.tsx
@@ -4,8 +4,7 @@ import { useEffect, useMemo } from 'react';
 import { Link } from 'react-router-dom';
 import { useIntercom } from 'react-use-intercom';
 
-import LogRocket from 'logrocket';
-import { CONTEXT_PATH, INTERCOM_APP_ID, IS_DOCKER_HOSTED, LOGROCKET_ID, REACT_APP_VERSION } from '../../../config';
+import { CONTEXT_PATH, INTERCOM_APP_ID, IS_DOCKER_HOSTED, REACT_APP_VERSION } from '../../../config';
 import { ROUTES } from '../../../constants/routes.enum';
 import {
   colors,
@@ -87,30 +86,6 @@ export function HeaderNav({ isIntercomOpened }: Props) {
     }
   }, [boot, currentUser, currentOrganization]);
 
-  useEffect(() => {
-    if (!LOGROCKET_ID) return;
-    if (currentUser && currentOrganization) {
-      let logrocketTraits;
-
-      if (currentUser?.email !== undefined) {
-        logrocketTraits = {
-          name: currentUser?.firstName + ' ' + currentUser?.lastName,
-          organizationId: currentOrganization._id,
-          organization: currentOrganization.name,
-          email: currentUser?.email ? currentUser?.email : ' ',
-        };
-      } else {
-        logrocketTraits = {
-          name: currentUser?.firstName + ' ' + currentUser?.lastName,
-          organizationId: currentOrganization._id,
-          organization: currentOrganization.name,
-        };
-      }
-
-      LogRocket.identify(currentUser?._id, logrocketTraits);
-    }
-  }, [currentUser, currentOrganization]);
-
   let themeTitle = 'Match System Appearance';
   if (themeStatus === 'dark') {
     themeTitle = 'Dark Theme';

diff --git a/apps/web/src/config/index.ts b/apps/web/src/config/index.ts
@@ -46,8 +46,6 @@ export const INTERCOM_APP_ID = window._env_.REACT_APP_INTERCOM_APP_ID || process
 
 export const CONTEXT_PATH = getContextPath(NovuComponentEnum.WEB);
 
-export const LOGROCKET_ID = (window._env_.REACT_APP_LOGROCKET_ID || process.env.REACT_APP_LOGROCKET_ID) ?? '';
-
 export const WEBHOOK_URL = isCypress
   ? window._env_.REACT_APP_WEBHOOK_URL || process.env.REACT_APP_WEBHOOK_URL || 'http://localhost:1341'
   : window._env_.REACT_APP_WEBHOOK_URL || process.env.REACT_APP_WEBHOOK_URL || 'http://localhost:3003';

diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml