improve sentry calls

spindle-app · Mar 4, 2024 · b3cdec9 · b3cdec9
1 parent 866a0a5
commit b3cdec9
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 12 deletions.
diff --git a/next.config.mjs b/next.config.mjs
@@ -29,9 +29,6 @@ if (process.env.NEXT_PUBLIC_SENTRY_DSN) {
       // Upload a larger set of source maps for prettier stack traces (increases build time)
       widenClientFileUpload: true,
 
-      // Transpiles SDK to be compatible with IE11 (increases bundle size)
-      transpileClientSDK: true,
-
       // Routes browser requests to Sentry through a Next.js rewrite to circumvent ad-blockers (increases server load)
       tunnelRoute: "/monitoring",
 

diff --git a/src/lib/errors/index.ts b/src/lib/errors/index.ts
@@ -14,5 +14,5 @@ export function captureApiError(
 }
 
 export function searchError(name: string, e: unknown) {
-  sentry.captureException(e, { tags: name });
+  sentry.captureException(e, { tags: { name } });
 }
diff --git a/src/server/sentry/index.ts b/src/server/sentry/index.ts
@@ -2,18 +2,21 @@ import { __API_ENV__ } from "~/lib/constants";
 
 import * as Sentry from "@sentry/nextjs";
 
-export function captureException(e, metadata) {
-  if (__API_ENV__ === "development") {
-    console.log("exception", e, metadata);
-  } else {
+export function captureException(
+  e: unknown,
+  metadata: Parameters<typeof Sentry.captureException>[1]
+) {
+  if (process.env.NEXT_PUBLIC_SENTRY_DSN) {
     Sentry.captureException(e, metadata);
+  } else {
+    console.log("exception", e, metadata);
   }
 }
 
-export function captureMessage(message, metadata) {
-  if (__API_ENV__ === "development") {
-    console.log("captureMessage", message, metadata);
+export function captureMessage(message: string, tags: Record<string, any>) {
+  if (process.env.NEXT_PUBLIC_SENTRY_DSN) {
+    Sentry.captureMessage(message, { tags });
   } else {
-    Sentry.captureMessage(message, { tags: metadata });
+    console.log("captureMessage", message, tags);
   }
 }