ReferenceError: name is not defined

Is there an existing issue for this?

• I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
• I have reviewed the documentation https://docs.sentry.io/
• I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?

Sentry Saas (sentry.io)

Which package are you using?

@sentry/remix

SDK Version

7.6.0

Framework Version

7.6.0

Link to Sentry event

https://sentry.io/organizations/syncwith/issues/3415220654/?project=5880203&query=is%3Aunresolved

Steps to Reproduce

I am trying out sentry-remix for the first time, and now when my app encounters a 404 it seems like our 404 page doesn’t get used, and I get an unexpected exception tracked to sentry

The error seems to be in:

https://github.com/getsentry/sentry-javascript/blob/master/packages/remix/src/utils/instrumentServer.ts

My app in root looks like:

function App() {
  const data = useLoaderData<RootLoaderData>();
  return (
    <Document data={data}>
      <Outlet />
      <footer></footer>
    </Document>
  );
}

export default withSentry(App);

Expected Result

I expected our normal 404 page to show, I think through the CatchBoundary

Actual Result

Error: name is not defined File “/Users/alex/dev/micro/webapp/node_modules/@sentry/remix/cjs/utils/instrumentServer.js”, line 65, col 40, in <anonymous> captureRemixServerException(err, name); File “/Users/alex/dev/micro/webapp/node_modules/@remix-run/server-runtime/server.js”, line 393, col 18, in handleDocumentRequest return await handleDocumentRequest(request.clone(), responseStatusCode, responseHeaders, entryContext); File “/Users/alex/dev/micro/webapp/node_modules/@remix-run/server-runtime/server.js”, line 49, col 18, in requestHandler response = await handleDocumentRequest({ File “/Users/alex/dev/micro/webapp/node_modules/@sentry/remix/cjs/utils/instrumentServer.js”, line 134, col 16, in <anonymous> var res = (await origRequestHandler.call(this, request, loadContext)) ; File “/Users/alex/dev/micro/webapp/node_modules/@remix-run/express/server.js”, line 41, col 22, in <anonymous> let response = await handleRequest(request, loadContext);

[alexblack]

@AbhiPrasad ok thanks, I reverted to 7.5.1 to avoid this issue of sentry throwing errors on 404 pages

[AbhiPrasad]

@alexblack We have a couple more remix fixes to make, and then we’ll make another release! End of week by latest.