add condition for not capturing static foles from next.js

last9 · May 16, 2024 · b0bdc6e · b0bdc6e
1 parent 47c238d
commit b0bdc6e
Showing 1 changed file with 18 additions and 12 deletions.
diff --git a/src/clients/nextjs.ts b/src/clients/nextjs.ts
@@ -52,22 +52,28 @@ export const instrumentNextjs = (
       ) as RequestMeta['match'];
       const parsedPath = requestMetaMatch?.definition.pathname;
 
-      counter?.inc({
-        path: parsedPath !== '' ? parsedPath : '/',
-        method: req.method ?? 'GET',
-        status: res.statusCode?.toString() ?? '500'
-        // ...(store?.labels ?? {}) -> // TODO: Implement dynamic labels
-      });
-
-      histogram?.observe(
-        {
+      if (
+        parsedPath &&
+        !parsedPath.startsWith('/_next/static/') &&
+        !parsedPath.startsWith('/favicon.ico')
+      ) {
+        counter?.inc({
           path: parsedPath !== '' ? parsedPath : '/',
           method: req.method ?? 'GET',
           status: res.statusCode?.toString() ?? '500'
           // ...(store?.labels ?? {}) -> // TODO: Implement dynamic labels
-        },
-        duration
-      );
+        });
+
+        histogram?.observe(
+          {
+            path: parsedPath !== '' ? parsedPath : '/',
+            method: req.method ?? 'GET',
+            status: res.statusCode?.toString() ?? '500'
+            // ...(store?.labels ?? {}) -> // TODO: Implement dynamic labels
+          },
+          duration
+        );
+      }
 
       return result;
     };