fix(nextjs): Don't write to res.end to fix next export

lforst · lforst · commit d9ca31b2f39b · 2023-01-07T16:21:56.000Z
diff --git a/packages/nextjs/src/config/wrappers/utils/responseEnd.ts b/packages/nextjs/src/config/wrappers/utils/responseEnd.ts
@@ -31,7 +31,8 @@ export function autoEndTransactionOnResponseEnd(transaction: Transaction, res: S
   };
 
   // Prevent double-wrapping
-  if (!(res.end as WrappedResponseEndMethod).__sentry_original__) {
+  // res.end may be undefined during build when using `next export` to statically export a Next.js app
+  if (res.end && !(res.end as WrappedResponseEndMethod).__sentry_original__) {
     fill(res, 'end', wrapEndMethod);
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,8 @@ export function autoEndTransactionOnResponseEnd(transaction: Transaction, res: S`
`31`	`31`	`};`
`32`	`32`
`33`	`33`	`// Prevent double-wrapping`
`34`		`- if (!(res.end as WrappedResponseEndMethod).__sentry_original__) {`
	`34`	+ // res.end may be undefined during build when using `next export` to statically export a Next.js app
	`35`	`+ if (res.end && !(res.end as WrappedResponseEndMethod).__sentry_original__) {`
`35`	`36`	`fill(res, 'end', wrapEndMethod);`
`36`	`37`	`}`
`37`	`38`	`}`