fix(nextjs): Prevent webpack 5 from crashing server (getsentry#3642)

AbhiPrasad · web-flow · commit 6c1b4bd71b46 · 2021-06-02T18:29:32.000+02:00
It seems that awaiting a flush() call is throwing errors in
Webpack 5. This crashes the server as the handler has already
sent a response at that point. This patch wraps the flush call
in a try, catch until we can figure out a better fix.
diff --git a/packages/nextjs/src/utils/handlers.ts b/packages/nextjs/src/utils/handlers.ts
@@ -81,7 +81,11 @@ export const withSentry = (handler: NextApiHandler): WrappedNextApiHandler => {
 
         transaction.finish();
       }
-      await flush(2000);
+      try {
+        await flush(2000);
+      } catch (e) {
+        // no-empty
+      }
     }
   };
 };

Original file line number	Diff line number	Diff line change
`@@ -81,7 +81,11 @@ export const withSentry = (handler: NextApiHandler): WrappedNextApiHandler => {`
`81`	`81`
`82`	`82`	`transaction.finish();`
`83`	`83`	`}`
`84`		`- await flush(2000);`
	`84`	`+ try {`
	`85`	`+ await flush(2000);`
	`86`	`+ } catch (e) {`
	`87`	`+ // no-empty`
	`88`	`+ }`
`85`	`89`	`}`
`86`	`90`	`};`
`87`	`91`	`};`