Use vercel log drain in merchant apps (#1657)

* Use vercel log drain * Add changeset
saleor · Dec 16, 2024 · bb269a5 · bb269a5
1 parent f62403e
commit bb269a5
Show file tree

Hide file tree

Showing 6 changed files with 80 additions and 34 deletions.
diff --git a/.changeset/empty-ligers-brush.md b/.changeset/empty-ligers-brush.md
@@ -0,0 +1,9 @@
+---
+"products-feed": patch
+"klaviyo": patch
+"cms-v2": patch
+"search": patch
+"smtp": patch
+---
+
+Added new `LoggerVercelTransport` support. It will help us send logs to our infrastructure without need of OTEL unstable logs API.
diff --git a/apps/cms-v2/src/logger.ts b/apps/cms-v2/src/logger.ts
@@ -9,14 +9,21 @@ if (process.env.NODE_ENV !== "production") {
 }
 
 if (typeof window === "undefined") {
-  import("@saleor/apps-logger/node").then(
-    async ({ attachLoggerOtelTransport, attachLoggerSentryTransport, LoggerContext }) => {
-      const loggerContext = await import("./logger-context").then((m) => m.loggerContext);
+  // Don't remove require - it's necessary for proper logger initialization
+  const {
+    attachLoggerSentryTransport,
+    attachLoggerVercelRuntimeTransport,
+  } = require("@saleor/apps-logger/node");
 
-      attachLoggerSentryTransport(rootLogger);
-      attachLoggerOtelTransport(rootLogger, packageJson.version, loggerContext);
-    },
-  );
+  attachLoggerSentryTransport(rootLogger);
+
+  if (process.env.NODE_ENV === "production") {
+    attachLoggerVercelRuntimeTransport(
+      rootLogger,
+      packageJson.version,
+      require("./logger-context").loggerContext,
+    );
+  }
 }
 
 export const createLogger = (name: string, params?: Record<string, unknown>) =>

diff --git a/apps/klaviyo/src/logger.ts b/apps/klaviyo/src/logger.ts
@@ -9,14 +9,21 @@ if (process.env.NODE_ENV !== "production") {
 }
 
 if (typeof window === "undefined") {
-  import("@saleor/apps-logger/node").then(
-    async ({ attachLoggerOtelTransport, attachLoggerSentryTransport }) => {
-      const loggerContext = await import("./logger-context").then((m) => m.loggerContext);
+  // Don't remove require - it's necessary for proper logger initialization
+  const {
+    attachLoggerSentryTransport,
+    attachLoggerVercelRuntimeTransport,
+  } = require("@saleor/apps-logger/node");
 
-      attachLoggerSentryTransport(rootLogger);
-      attachLoggerOtelTransport(rootLogger, packageJson.version, loggerContext);
-    },
-  );
+  attachLoggerSentryTransport(rootLogger);
+
+  if (process.env.NODE_ENV === "production") {
+    attachLoggerVercelRuntimeTransport(
+      rootLogger,
+      packageJson.version,
+      require("./logger-context").loggerContext,
+    );
+  }
 }
 
 export const createLogger = (name: string, params?: Record<string, unknown>) =>

diff --git a/apps/products-feed/src/logger.ts b/apps/products-feed/src/logger.ts
@@ -9,14 +9,21 @@ if (process.env.NODE_ENV !== "production") {
 }
 
 if (typeof window === "undefined") {
-  import("@saleor/apps-logger/node").then(
-    async ({ attachLoggerOtelTransport, attachLoggerSentryTransport }) => {
-      const loggerContext = await import("./logger-context").then((m) => m.loggerContext);
+  // Don't remove require - it's necessary for proper logger initialization
+  const {
+    attachLoggerSentryTransport,
+    attachLoggerVercelRuntimeTransport,
+  } = require("@saleor/apps-logger/node");
 
-      attachLoggerSentryTransport(rootLogger);
-      attachLoggerOtelTransport(rootLogger, packageJson.version, loggerContext);
-    },
-  );
+  attachLoggerSentryTransport(rootLogger);
+
+  if (process.env.NODE_ENV === "production") {
+    attachLoggerVercelRuntimeTransport(
+      rootLogger,
+      packageJson.version,
+      require("./logger-context").loggerContext,
+    );
+  }
 }
 
 export const createLogger = (name: string, params?: Record<string, unknown>) =>

diff --git a/apps/search/src/lib/logger.ts b/apps/search/src/lib/logger.ts
@@ -10,12 +10,21 @@ if (process.env.NODE_ENV !== "production") {
 }
 
 if (typeof window === "undefined") {
-  import("@saleor/apps-logger/node").then(
-    ({ attachLoggerOtelTransport, attachLoggerSentryTransport }) => {
-      attachLoggerSentryTransport(rootLogger);
-      attachLoggerOtelTransport(rootLogger, packageJson.version);
-    },
-  );
+  // Don't remove require - it's necessary for proper logger initialization
+  const {
+    attachLoggerSentryTransport,
+    attachLoggerVercelRuntimeTransport,
+  } = require("@saleor/apps-logger/node");
+
+  attachLoggerSentryTransport(rootLogger);
+
+  if (process.env.NODE_ENV === "production") {
+    attachLoggerVercelRuntimeTransport(
+      rootLogger,
+      packageJson.version,
+      require("./logger-context").loggerContext,
+    );
+  }
 }
 
 export const createLogger = (name: string, params?: Record<string, unknown>) =>

diff --git a/apps/smtp/src/logger.ts b/apps/smtp/src/logger.ts
@@ -9,14 +9,21 @@ if (process.env.NODE_ENV !== "production") {
 }
 
 if (typeof window === "undefined") {
-  import("@saleor/apps-logger/node").then(
-    async ({ attachLoggerOtelTransport, attachLoggerSentryTransport }) => {
-      const loggerContext = await import("./logger-context").then((m) => m.loggerContext);
+  // Don't remove require - it's necessary for proper logger initialization
+  const {
+    attachLoggerSentryTransport,
+    attachLoggerVercelRuntimeTransport,
+  } = require("@saleor/apps-logger/node");
 
-      attachLoggerSentryTransport(rootLogger);
-      attachLoggerOtelTransport(rootLogger, packageJson.version, loggerContext);
-    },
-  );
+  attachLoggerSentryTransport(rootLogger);
+
+  if (process.env.NODE_ENV === "production") {
+    attachLoggerVercelRuntimeTransport(
+      rootLogger,
+      packageJson.version,
+      require("./logger-context").loggerContext,
+    );
+  }
 }
 
 export const createLogger = (name: string, params?: Record<string, unknown>) =>