chore(nextjs): Remove telemetry event from clerkMiddleware() (#5501)

Author: brkalow

.changeset/spotty-schools-juggle.md

@@ -0,0 +1,7 @@
+---
+'@clerk/nextjs': patch
+'@clerk/astro': patch
+'@clerk/nuxt': patch
+---
+
+Remove telemtry event from `clerkMiddleware()`.

packages/astro/src/server/clerk-middleware.ts

@@ -3,7 +3,6 @@ import type { AuthenticateRequestOptions, ClerkRequest, RedirectFun, RequestStat
 import { AuthStatus, constants, createClerkRequest, createRedirect } from '@clerk/backend/internal';
 import { isDevelopmentFromPublishableKey, isDevelopmentFromSecretKey } from '@clerk/shared/keys';
 import { isHttpOrHttps } from '@clerk/shared/proxy';
-import { eventMethodCalled } from '@clerk/shared/telemetry';
 import { handleValueOrFn } from '@clerk/shared/utils';
 import type { APIContext } from 'astro';
 
@@ -68,14 +67,6 @@ export const clerkMiddleware: ClerkMiddleware = (...args: unknown[]): any => {
 
     const clerkRequest = createClerkRequest(context.request);
 
-    clerkClient(context).telemetry.record(
-      eventMethodCalled('clerkMiddleware', {
-        handler: Boolean(handler),
-        satellite: Boolean(options.isSatellite),
-        proxy: Boolean(options.proxyUrl),
-      }),
-    );
-
     const requestState = await clerkClient(context).authenticateRequest(
       clerkRequest,
       createAuthenticateRequestOptions(clerkRequest, options, context),

packages/nextjs/src/server/clerkMiddleware.ts

@@ -1,7 +1,6 @@
 import type { AuthObject, ClerkClient } from '@clerk/backend';
 import type { AuthenticateRequestOptions, ClerkRequest, RedirectFun, RequestState } from '@clerk/backend/internal';
 import { AuthStatus, constants, createClerkRequest, createRedirect } from '@clerk/backend/internal';
-import { eventMethodCalled } from '@clerk/shared/telemetry';
 import { notFound as nextjsNotFound } from 'next/navigation';
 import type { NextMiddleware, NextRequest } from 'next/server';
 import { NextResponse } from 'next/server';
@@ -128,14 +127,6 @@ export const clerkMiddleware = ((...args: unknown[]): NextMiddleware | NextMiddl
       clerkMiddlewareRequestDataStore.set('requestData', options);
       const resolvedClerkClient = await clerkClient();
 
-      resolvedClerkClient.telemetry.record(
-        eventMethodCalled('clerkMiddleware', {
-          handler: Boolean(handler),
-          satellite: Boolean(options.isSatellite),
-          proxy: Boolean(options.proxyUrl),
-        }),
-      );
-
       if (options.debug) {
         logger.enable();
       }

packages/nuxt/src/runtime/server/clerkMiddleware.ts

@@ -1,6 +1,5 @@
 import type { AuthenticateRequestOptions } from '@clerk/backend/internal';
 import { AuthStatus, constants } from '@clerk/backend/internal';
-import { eventMethodCalled } from '@clerk/shared/telemetry';
 import type { EventHandler } from 'h3';
 import { createError, eventHandler, setResponseHeader } from 'h3';
 
@@ -80,14 +79,6 @@ export const clerkMiddleware: ClerkMiddleware = (...args: unknown[]) => {
   return eventHandler(async event => {
     const clerkRequest = toWebRequest(event);
 
-    clerkClient(event).telemetry.record(
-      eventMethodCalled('clerkMiddleware', {
-        handler: Boolean(handler),
-        satellite: Boolean(options.isSatellite),
-        proxy: Boolean(options.proxyUrl),
-      }),
-    );
-
     const requestState = await clerkClient(event).authenticateRequest(clerkRequest, options);
 
     const locationHeader = requestState.headers.get(constants.Headers.Location);