chore(logging): uninstall pino and remove all logs (#1740)

entur · Nov 20, 2024 · b6163ea · b6163ea
1 parent 0a17d79
commit b6163ea
Show file tree

Hide file tree

Showing 9 changed files with 5 additions and 310 deletions.
diff --git a/tavla/app/(admin)/utils/serverErrorLogger.ts b/tavla/app/(admin)/utils/serverErrorLogger.ts
diff --git a/tavla/app/error.tsx b/tavla/app/error.tsx
@@ -4,27 +4,9 @@ import { Button } from '@entur/button'
 import { Heading3 } from '@entur/typography'
 import Image from 'next/image'
 import Link from 'next/link'
-import { useEffect } from 'react'
 import BeaverIllustration from 'assets/illustrations/Beaver.png'
-import { logServerError } from './(admin)/utils/serverErrorLogger'
-
-export default function Error({
-    error,
-}: {
-    error: Error & { digest?: string }
-}) {
-    useEffect(() => {
-        const errorDetails = {
-            message: error.message,
-            stack: error.stack,
-            name: error.name,
-            cause: error.cause,
-            digest: error.digest,
-        }
-
-        logServerError(errorDetails)
-    }, [error])
 
+export default function Error() {
     return (
         <main className="container pb-10 flex flex-col items-center">
             <Heading3>Au da! Noe gikk galt!</Heading3>

diff --git a/tavla/app/global-error.tsx b/tavla/app/global-error.tsx
@@ -4,26 +4,10 @@ import { Button } from '@entur/button'
 import { Heading3 } from '@entur/typography'
 import Image from 'next/image'
 import Link from 'next/link'
-import { useEffect } from 'react'
-import BeaverIllustration from 'assets/illustrations/Beaver.png'
-import { logServerError } from './(admin)/utils/serverErrorLogger'
 
-export default function Error({
-    error,
-}: {
-    error: Error & { digest?: string }
-}) {
-    useEffect(() => {
-        const errorDetails = {
-            message: error.message,
-            stack: error.stack,
-            name: error.name,
-            cause: error.cause,
-            digest: error.digest,
-        }
-        logServerError(errorDetails)
-    }, [error])
+import BeaverIllustration from 'assets/illustrations/Beaver.png'
 
+export default function Error() {
     return (
         <html>
             <body>

diff --git a/tavla/middleware.ts b/tavla/middleware.ts
diff --git a/tavla/next.config.js b/tavla/next.config.js
@@ -20,9 +20,6 @@ const nextConfig = {
             },
         ],
     },
-    experimental: {
-        serverComponentsExternalPackages: ['pino', 'pino-pretty'],
-    },
 }
 
 module.exports = async (phase, { defaultConfig }) => {

diff --git a/tavla/package.json b/tavla/package.json
@@ -51,8 +51,6 @@
         "lodash": "4.17.21",
         "nanoid": "5.0.8",
         "next": "14.2.4",
-        "pino": "9.5.0",
-        "pino-pretty": "11.3.0",
         "posthog-js": "1.176.0",
         "react": "18.3.1",
         "react-dom": "18.3.1",

diff --git a/tavla/pages/_error.tsx b/tavla/pages/_error.tsx
@@ -4,11 +4,8 @@ import { Button } from '@entur/button'
 import { Heading3 } from '@entur/typography'
 import Image from 'next/image'
 import Link from 'next/link'
-import { logger } from 'utils/logger'
 import BeaverIllustration from 'assets/illustrations/Beaver.png'
-import { NextPageContext } from 'next'
 
-const log = logger.child({ module: 'pagesErrorHandler' })
 function Error() {
     return (
         <main className="container pb-10 flex flex-col items-center">
@@ -26,14 +23,4 @@ function Error() {
     )
 }
 
-Error.getInitialProps = ({ res, err }: NextPageContext) => {
-    const statusCode = res ? res.statusCode : err ? err.statusCode : 404
-    log.error({
-        statusCode: statusCode,
-        error: err,
-        res: res,
-    })
-    return { statusCode }
-}
-
 export default Error
diff --git a/tavla/src/Shared/utils/logger.ts b/tavla/src/Shared/utils/logger.ts