fix: notify activity called for any proxied request (#7356)

VitaliyR · web-flow · commit e6f0d1b4645b · 2025-06-23T11:50:22.000-04:00
diff --git a/src/utils/proxy.ts b/src/utils/proxy.ts
@@ -859,15 +859,13 @@ const onRequest = async (
   }
 
   const maybeNotifyActivity = () => {
-    if (req.method === 'GET' && api && process.env.NETLIFY_DEV_SERVER_ID) {
+    if (api && process.env.NETLIFY_DEV_SERVER_ID && (req.method === 'GET' || req.url?.startsWith('/.ntlfy-dev/'))) {
       notifyActivity(api, siteInfo.id, process.env.NETLIFY_DEV_SERVER_ID)
     }
   }
 
   if (match) {
-    if (!isExternal(match)) {
-      maybeNotifyActivity()
-    }
+    maybeNotifyActivity()
 
     // We don't want to generate an ETag for 3xx redirects.
     // @ts-expect-error TS(7031) FIXME: Binding element 'statusCode' implicitly has an 'an... Remove this comment to see the full error message

Original file line number	Diff line number	Diff line change
`@@ -859,15 +859,13 @@ const onRequest = async (`
`859`	`859`	`}`
`860`	`860`
`861`	`861`	`const maybeNotifyActivity = () => {`
`862`		`- if (req.method === 'GET' && api && process.env.NETLIFY_DEV_SERVER_ID) {`
	`862`	`+ if (api && process.env.NETLIFY_DEV_SERVER_ID && (req.method === 'GET' \|\| req.url?.startsWith('/.ntlfy-dev/'))) {`
`863`	`863`	`notifyActivity(api, siteInfo.id, process.env.NETLIFY_DEV_SERVER_ID)`
`864`	`864`	`}`
`865`	`865`	`}`
`866`	`866`
`867`	`867`	`if (match) {`
`868`		`- if (!isExternal(match)) {`
`869`		`- maybeNotifyActivity()`
`870`		`- }`
	`868`	`+ maybeNotifyActivity()`
`871`	`869`
`872`	`870`	`// We don't want to generate an ETag for 3xx redirects.`
`873`	`871`	`// @ts-expect-error TS(7031) FIXME: Binding element 'statusCode' implicitly has an 'an... Remove this comment to see the full error message`