chore: remove trailing slash logging (revert #2743) (#2805)

mrstork · web-flow · commit f978b8a3a810 · 2025-04-01T11:13:59.000-04:00
diff --git a/src/run/handlers/server.ts b/src/run/handlers/server.ts
@@ -131,7 +131,7 @@ export default async (
       })
     }
 
-    setCacheControlHeaders(response, request, requestContext, nextConfig)
+    setCacheControlHeaders(response, request, requestContext)
     setCacheTagsHeaders(response.headers, requestContext)
     setVaryHeaders(response.headers, request, nextConfig)
     setCacheStatusHeader(response.headers, nextCache)
diff --git a/src/run/headers.ts b/src/run/headers.ts
@@ -3,7 +3,7 @@ import type { NextConfigComplete } from 'next/dist/server/config-shared.js'
 
 import type { NetlifyCachedRouteValue, NetlifyCacheHandlerValue } from '../shared/cache-types.cjs'
 
-import { getLogger, RequestContext } from './handlers/request-context.cjs'
+import { RequestContext } from './handlers/request-context.cjs'
 import { recordWarning } from './handlers/tracer.cjs'
 import { getMemoizedKeyValueStoreBackedByRegionalBlobStore } from './storage/storage.cjs'
 
@@ -200,7 +200,6 @@ export const setCacheControlHeaders = (
   { headers, status }: Response,
   request: Request,
   requestContext: RequestContext,
-  nextConfig: NextConfigComplete,
 ) => {
   if (
     typeof requestContext.routeHandlerRevalidate !== 'undefined' &&
@@ -213,13 +212,6 @@ export const setCacheControlHeaders = (
     return
   }
 
-  // temporary diagnostic to evaluate number of trailing slash redirects
-  if (status === 308 && request.url.endsWith('/') !== nextConfig.trailingSlash) {
-    getLogger()
-      .withFields({ trailingSlash: nextConfig.trailingSlash, location: headers.get('location') })
-      .log('NetlifyHeadersHandler.trailingSlashRedirect')
-  }
-
   const cacheControl = headers.get('cache-control')
   if (status === 404) {
     if (request.url.endsWith('.php')) {

Original file line number	Diff line number	Diff line change
`@@ -131,7 +131,7 @@ export default async (`
`131`	`131`	`})`
`132`	`132`	`}`
`133`	`133`
`134`		`- setCacheControlHeaders(response, request, requestContext, nextConfig)`
	`134`	`+ setCacheControlHeaders(response, request, requestContext)`
`135`	`135`	`setCacheTagsHeaders(response.headers, requestContext)`
`136`	`136`	`setVaryHeaders(response.headers, request, nextConfig)`
`137`	`137`	`setCacheStatusHeader(response.headers, nextCache)`